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 411095 d2a4436a43499b7fbc2e47600cfbea3740b896a0
parent 411094 6e0d544cc5cadbdb3090c60b532ba1b95fbb32cc
child 411096 b42d50cafb154e3e50fdde3ca853635a000cb219
child 411238 4da23071bc0ed686316d2dbb3a537439546ca04a
push id7391
push usermtabara@mozilla.com
push dateMon, 12 Jun 2017 13:08:53 +0000
treeherdermozilla-beta@2191d7f87e2e [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.