Bug 1266163 - Replace FENNEC_READER_VIEW_BUTTON histogram with UI telemetry. r=mfinkle,barbara draft
authorMargaret Leibovic <margaret.leibovic@gmail.com>
Wed, 20 Apr 2016 15:22:46 -0400
changeset 355405 a474fb10ef8e197313debb970640f1243b2dfca1
parent 355404 df4e550128b976a664216e57f78f80c36d78f275
child 519187 2c671e32bd19618ae463810e69c444c10ea81c92
push id16276
push usermleibovic@mozilla.com
push dateFri, 22 Apr 2016 14:46:46 +0000
reviewersmfinkle, barbara
bugs1266163
milestone48.0a1
Bug 1266163 - Replace FENNEC_READER_VIEW_BUTTON histogram with UI telemetry. r=mfinkle,barbara MozReview-Commit-ID: 1guxvXp6loC
mobile/android/chrome/content/Reader.js
toolkit/components/telemetry/Histograms.json
--- a/mobile/android/chrome/content/Reader.js
+++ b/mobile/android/chrome/content/Reader.js
@@ -17,29 +17,16 @@ var Reader = {
   STATUS_FETCH_FAILED_UNSUPPORTED_FORMAT: 3,
   STATUS_FETCHED_ARTICLE: 4,
 
   get _hasUsedToolbar() {
     delete this._hasUsedToolbar;
     return this._hasUsedToolbar = Services.prefs.getBoolPref("reader.has_used_toolbar");
   },
 
-  get _buttonHistogram() {
-    delete this._buttonHistogram;
-    return this._buttonHistogram = Services.telemetry.getHistogramById("FENNEC_READER_VIEW_BUTTON");
-  },
-
-  // Values for "FENNEC_READER_VIEW_BUTTON" histogram.
-  _buttonHistogramValues: {
-    HIDDEN: 0,
-    SHOWN: 1,
-    TAP_ENTER: 2,
-    TAP_EXIT: 3
-  },
-
   /**
    * BackPressListener (listeners / ReaderView Ids).
    */
   _backPressListeners: [],
   _backPressViewIds: [],
 
   /**
    * Set a backPressListener for this tabId / ReaderView Id pair.
@@ -168,20 +155,20 @@ var Reader = {
       let url = browser.currentURI.spec;
       if (url.startsWith("about:reader")) {
         let originalURL = ReaderMode.getOriginalUrl(url);
         if (!originalURL) {
           Cu.reportError("Error finding original URL for about:reader URL: " + url);
         } else {
           browser.loadURI(originalURL);
         }
-        Reader._buttonHistogram.add(Reader._buttonHistogramValues.TAP_EXIT);
+        UITelemetry.addEvent("action.1", "button", "reader_exit");
       } else {
         browser.messageManager.sendAsyncMessage("Reader:ParseDocument", { url: url });
-        Reader._buttonHistogram.add(Reader._buttonHistogramValues.TAP_ENTER);
+        UITelemetry.addEvent("action.1", "button", "reader_enter");
       }
     },
   },
 
   updatePageAction: function(tab) {
     if (!tab.getActive()) {
       return;
     }
@@ -209,19 +196,19 @@ var Reader = {
       return;
     }
 
     // Only stop a reader session if the foreground viewer is not visible.
     UITelemetry.stopSession("reader.1", "", null);
 
     if (browser.isArticle) {
       showPageAction("drawable://reader", Strings.reader.GetStringFromName("readerView.enter"));
-      this._buttonHistogram.add(this._buttonHistogramValues.SHOWN);
+      UITelemetry.addEvent("show.1", "button", "reader_available");
     } else {
-      this._buttonHistogram.add(this._buttonHistogramValues.HIDDEN);
+      UITelemetry.addEvent("show.1", "button", "reader_unavailable");
     }
   },
 
   /**
    * Downloads and caches content for a reading list item with a given URL and id.
    */
   _fetchContent: function(url, id) {
     this._downloadAndCacheArticle(url).then(article => {
--- a/toolkit/components/telemetry/Histograms.json
+++ b/toolkit/components/telemetry/Histograms.json
@@ -9399,23 +9399,16 @@
   },
   "READER_MODE_DOWNLOAD_RESULT" : {
     "expires_in_version": "50",
     "alert_emails": ["mleibovic@mozilla.com"],
     "kind": "enumerated",
     "n_values": 5,
     "description": "The result of trying to download a document to show in reader view (0=Success, 1=Error XHR, 2=Error no document)"
   },
-  "FENNEC_READER_VIEW_BUTTON" : {
-    "expires_in_version": "50",
-    "alert_emails": ["mobile-frontend@mozilla.com"],
-    "kind": "enumerated",
-    "n_values": 10,
-    "description": "Bug 1219240: Measures user interaction with the reader view button (0=Button hidden, 1=Button shown, 2=Tap to enter reader view, 3=Tap to exit reader view, 4=Long tap)"
-  },
   "FENNEC_LOAD_SAVED_PAGE": {
     "expires_in_version": "50",
     "alert_emails": ["mobile-frontend@mozilla.com"],
     "kind": "enumerated",
     "n_values": 10,
     "description": "How often users load saved items when online/offline (0=RL online, 1=RL offline, 2=BM online, 3=BM offline)",
     "bug_numbers": [1243387]
   },