Bug 1342490 - DevTools users are temporarily qualified for e10s. r=felipe draft
authorJ. Ryan Stinnett <jryans@gmail.com>
Fri, 24 Feb 2017 14:36:05 -0600
changeset 492227 4112ba261d417087979012c984d53327e4396a08
parent 492226 3a7d1e7f93bd3bf845fe24666fb12c1d98ac774d
child 492228 fffb4f1a947ea81e5ee28a5bc1ec7488898f50e5
push id47559
push userbmo:jryans@gmail.com
push dateThu, 02 Mar 2017 22:39:35 +0000
reviewersfelipe
bugs1342490
milestone54.0a1
Bug 1342490 - DevTools users are temporarily qualified for e10s. r=felipe 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 "";
 }