Bug 1342490 - DevTools users are temporarily qualified for e10s. r=Felipe, a=lizzard
authorJ. Ryan Stinnett <jryans@gmail.com>
Fri, 24 Feb 2017 14:36:05 -0600
changeset 378879 f8757303ef9aad5b0c0717316a1c97c66bfe6090
parent 378878 0ba4b0ec4c6ef87f37564260b6c4c556e8b7d987
child 378880 44337c730a8a171bfbfcd54460b23ac656baff88
push id1419
push userjlund@mozilla.com
push dateMon, 10 Apr 2017 20:44:07 +0000
treeherdermozilla-release@5e6801b73ef6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersFelipe, lizzard
bugs1342490
milestone53.0a2
Bug 1342490 - DevTools users are temporarily qualified for e10s. r=Felipe, a=lizzard MozReview-Commit-ID: 4CZgSMcF1NT
browser/extensions/e10srollout/bootstrap.js
--- a/browser/extensions/e10srollout/bootstrap.js
+++ b/browser/extensions/e10srollout/bootstrap.js
@@ -181,10 +181,20 @@ function getTemporaryDisqualification() 
 
 /* If this function returns a non-empty string, it
  * means that this particular user should be temporarily
  * qualified due to some particular reason.
  * If a user shouldn't be qualified, then an empty
  * string must be returned.
  */
 function getTemporaryQualification() {
+  // Whenever the DevTools toolbox is opened for the first time in a release, it
+  // records this fact in the following pref as part of the DevTools telemetry
+  // system.  If this pref is set, then it means the user has opened DevTools at
+  // some point in time.
+  const PREF_OPENED_DEVTOOLS = "devtools.telemetry.tools.opened.version";
+  let hasOpenedDevTools = Preferences.isSet(PREF_OPENED_DEVTOOLS);
+  if (hasOpenedDevTools) {
+    return "devtools";
+  }
+
   return "";
 }