Bug 1267837 - Update grid item telemetry to indicate whether an item is pinned. r=mfinkle draft
authorMargaret Leibovic <margaret.leibovic@gmail.com>
Wed, 11 May 2016 11:40:49 -0400
changeset 365934 b901745bd6ae2b26690491be9cbe8020d85a2931
parent 364847 cbeb65bea19cef6b91ef50e4e38b473630c984d6
child 520653 95b773437ba1fe4c6702aed9b3854cec17561418
push id17847
push usermleibovic@mozilla.com
push dateWed, 11 May 2016 15:41:21 +0000
reviewersmfinkle
bugs1267837
milestone49.0a1
Bug 1267837 - Update grid item telemetry to indicate whether an item is pinned. r=mfinkle MozReview-Commit-ID: 9Et4AhCzAr9
mobile/android/base/java/org/mozilla/gecko/home/TopSitesPanel.java
--- a/mobile/android/base/java/org/mozilla/gecko/home/TopSitesPanel.java
+++ b/mobile/android/base/java/org/mozilla/gecko/home/TopSitesPanel.java
@@ -211,17 +211,23 @@ public class TopSitesPanel extends HomeF
                 if (type != TopSites.TYPE_BLANK) {
                     if (mUrlOpenListener != null) {
                         final TelemetryContract.Method method;
                         if (type == TopSites.TYPE_SUGGESTED) {
                             method = TelemetryContract.Method.SUGGESTION;
                         } else {
                             method = TelemetryContract.Method.GRID_ITEM;
                         }
-                        Telemetry.sendUIEvent(TelemetryContract.Event.LOAD_URL, method, Integer.toString(position));
+
+                        String extra = Integer.toString(position);
+                        if (type == TopSites.TYPE_PINNED) {
+                            extra += "-pinned";
+                        }
+
+                        Telemetry.sendUIEvent(TelemetryContract.Event.LOAD_URL, method, extra);
 
                         mUrlOpenListener.onUrlOpen(url, EnumSet.noneOf(OnUrlOpenListener.Flags.class));
                     }
                 } else {
                     if (mEditPinnedSiteListener != null) {
                         mEditPinnedSiteListener.onEditPinnedSite(position, "");
                     }
                 }