Bug 1008386 - Add thumbnail position to griditem loadurl probe r=lucasr a=sylvestre
authorMark Finkle <mfinkle@mozilla.com>
Mon, 12 May 2014 18:16:44 -0400
changeset 200297 47540d80872aaecfd8e79c08a754a475551777f8
parent 200296 331db2dde15b1c1e2c8da83f61656089a853b372
child 200298 894a8aef0e462f66f7f15de3305529a40272f951
push id486
push userasasaki@mozilla.com
push dateMon, 14 Jul 2014 18:39:42 +0000
treeherdermozilla-release@d33428174ff1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerslucasr, sylvestre
bugs1008386
milestone31.0a2
Bug 1008386 - Add thumbnail position to griditem loadurl probe r=lucasr a=sylvestre
mobile/android/base/home/TopSitesGridView.java
--- a/mobile/android/base/home/TopSitesGridView.java
+++ b/mobile/android/base/home/TopSitesGridView.java
@@ -107,17 +107,17 @@ public class TopSitesGridView extends Gr
 
                 // Decode "user-entered" URLs before loading them.
                 String url = HomeFragment.decodeUserEnteredUrl(row.getUrl());
 
                 // If the url is empty, the user can pin a site.
                 // If not, navigate to the page given by the url.
                 if (!TextUtils.isEmpty(url)) {
                     if (mUrlOpenListener != null) {
-                        Telemetry.sendUIEvent(TelemetryContract.Event.LOAD_URL, TelemetryContract.Method.GRID_ITEM);
+                        Telemetry.sendUIEvent(TelemetryContract.Event.LOAD_URL, TelemetryContract.Method.GRID_ITEM, Integer.toString(position));
 
                         mUrlOpenListener.onUrlOpen(url, EnumSet.noneOf(OnUrlOpenListener.Flags.class));
                     }
                 } else {
                     if (mEditPinnedSiteListener != null) {
                         mEditPinnedSiteListener.onEditPinnedSite(position, "");
                     }
                 }