Bug 1235061 - Create histogram probe to measure how often users have zoomed view enabled/disabled. r=mfinkle draft
authorMargaret Leibovic <margaret.leibovic@gmail.com>
Thu, 24 Dec 2015 13:58:21 -0500
changeset 317627 46fed5625f005793138bbe4288b6d92e5237ee51
parent 317626 506b5a07ebdf96d96bb9c84bd9c889d4fd466ef3
child 512326 482a22d37c1e6de63c6ddcfb32066c849eea378e
push id8728
push usermleibovic@mozilla.com
push dateThu, 24 Dec 2015 18:58:43 +0000
reviewersmfinkle
bugs1235061
milestone46.0a1
Bug 1235061 - Create histogram probe to measure how often users have zoomed view enabled/disabled. r=mfinkle
mobile/android/chrome/content/browser.js
toolkit/components/telemetry/Histograms.json
--- a/mobile/android/chrome/content/browser.js
+++ b/mobile/android/chrome/content/browser.js
@@ -595,20 +595,26 @@ var BrowserApp = {
       InitLater(() => Cu.import("resource://gre/modules/NotificationDB.jsm"));
       InitLater(() => Cu.import("resource://gre/modules/Payment.jsm"));
 
       InitLater(() => Services.obs.notifyObservers(window, "browser-delayed-startup-finished", ""));
       InitLater(() => Messaging.sendRequest({ type: "Gecko:DelayedStartup" }));
 
       if (AppConstants.NIGHTLY_BUILD) {
         InitLater(() => ShumwayUtils.init(), window, "ShumwayUtils");
-        InitLater(() => Telemetry.addData("FENNEC_TRACKING_PROTECTION_STATE", parseInt(BrowserApp.getTrackingProtectionState())));
         InitLater(() => WebcompatReporter.init());
       }
 
+      // Collect telemetry data.
+      // We do this at startup because we want to move away from "gather-telemetry" (bug 1127907)
+      InitLater(() => {
+        Telemetry.addData("FENNEC_TRACKING_PROTECTION_STATE", parseInt(BrowserApp.getTrackingProtectionState()));
+        Telemetry.addData("ZOOMED_VIEW_ENABLED", Services.prefs.getBoolPref("ui.zoomedview.enabled"));
+      });
+
       InitLater(() => LightWeightThemeWebInstaller.init());
       InitLater(() => SpatialNavigation.init(BrowserApp.deck, null), window, "SpatialNavigation");
       InitLater(() => CastingApps.init(), window, "CastingApps");
       InitLater(() => Services.search.init(), Services, "search");
       InitLater(() => DownloadNotifications.init(), window, "DownloadNotifications");
 
       if (AppConstants.MOZ_SAFE_BROWSING) {
         // Bug 778855 - Perf regression if we do this here. To be addressed in bug 779008.
--- a/toolkit/components/telemetry/Histograms.json
+++ b/toolkit/components/telemetry/Histograms.json
@@ -8197,16 +8197,23 @@
   },
   "TAP_TO_LOAD_ENABLED": {
     "expires_in_version": "50",
     "kind": "enumerated",
     "n_values": 3,
     "description": "Whether or not a user has tap-to-load enabled.",
     "bug_numbers": [1208167]
   },
+  "ZOOMED_VIEW_ENABLED": {
+    "expires_in_version": "50",
+    "kind": "boolean",
+    "description": "Whether or not a user has the zoomed view (a.k.a. \"Magnify small areas\") enabled.",
+    "alert_emails": ["mleibovic@mozilla.com"],
+    "bug_numbers": [1235061]
+  },
   "TRACKING_PROTECTION_ENABLED": {
     "expires_in_version": "never",
     "kind": "boolean",
     "description": "Whether or not a session has tracking protection enabled"
   },
   "TRACKING_PROTECTION_PBM_DISABLED": {
     "expires_in_version": "60",
     "kind": "boolean",