Bug 1369028 - Disable the extra telemetry reporting from follow-on search until the perf issues are fixed. r=past
authorMark Banner <standard8@mozilla.com>
Thu, 08 Jun 2017 09:45:54 +0100
changeset 362974 d2a4436a43499b7fbc2e47600cfbea3740b896a0
parent 362973 6e0d544cc5cadbdb3090c60b532ba1b95fbb32cc
child 362975 4da23071bc0ed686316d2dbb3a537439546ca04a
child 363037 b42d50cafb154e3e50fdde3ca853635a000cb219
push id91215
push usermbanner@mozilla.com
push dateThu, 08 Jun 2017 14:58:00 +0000
treeherdermozilla-inbound@d2a4436a4349 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspast
bugs1369028
milestone55.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1369028 - Disable the extra telemetry reporting from follow-on search until the perf issues are fixed. r=past MozReview-Commit-ID: 4N7kMeGii1q
browser/extensions/followonsearch/bootstrap.js
--- a/browser/extensions/followonsearch/bootstrap.js
+++ b/browser/extensions/followonsearch/bootstrap.js
@@ -162,22 +162,24 @@ var cohortManager = {
       if (prefOverride in REPORTING_THRESHOLD) {
         updateChannel = prefOverride;
       } else {
         // Don't enable, we don't know about the channel, and it isn't overriden.
         return;
       }
     }
 
-    if (cohortSample <= REPORTING_THRESHOLD[updateChannel]) {
-      log("Enabling telemetry for user");
-      this.enableForUser = true;
-    } else {
-      log("Not enabling telemetry for user - outside threshold.");
-    }
+    log("Not enabling extra telemetry due to bug 1371198");
+
+    // if (cohortSample <= REPORTING_THRESHOLD[updateChannel]) {
+    //   log("Enabling telemetry for user");
+    //   this.enableForUser = true;
+    // } else {
+    //   log("Not enabling telemetry for user - outside threshold.");
+    // }
   },
 };
 
 /**
  * Called when the add-on is installed.
  *
  * @param {Object} data Data about the add-on.
  * @param {Number} reason Indicates why the extension is being installed.