Bug 1357959 - Put people who forced into e10s into the multi test group. r=Felipe, a=gchang
authorBlake Kaplan <mrbkap@gmail.com>
Thu, 20 Apr 2017 17:35:43 -0700
changeset 396018 7f717abab712539d4134e4ce6bc23c63019fd102
parent 396017 40bd1b4eb78c562a91b7b80e9200eaeb23867245
child 396019 a236801aa0225218df1f74cc365bd68b6305a5db
push id1468
push userasasaki@mozilla.com
push dateMon, 05 Jun 2017 19:31:07 +0000
treeherdermozilla-release@0641fc6ee9d1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersFelipe, gchang
bugs1357959
milestone54.0
Bug 1357959 - Put people who forced into e10s into the multi test group. r=Felipe, a=gchang MozReview-Commit-ID: 3eAkCW8MtRf
browser/extensions/e10srollout/bootstrap.js
--- a/browser/extensions/e10srollout/bootstrap.js
+++ b/browser/extensions/e10srollout/bootstrap.js
@@ -122,16 +122,17 @@ function defineCohort() {
 
   let inMultiExperiment = false;
   if (userOptedOut.e10s || userOptedOut.multi) {
     // If we detected that the user opted out either for multi or e10s, then
     // the proper prefs must already be set.
     setCohort("optedOut");
   } else if (userOptedIn.e10s) {
     setCohort("optedIn");
+    inMultiExperiment = true;
   } else if (temporaryDisqualification != "") {
     // Users who are disqualified by the backend (from multiprocessBlockPolicy)
     // can be put into either the test or control groups, because e10s will
     // still be denied by the backend, which is useful so that the E10S_STATUS
     // telemetry probe can be correctly set.
 
     // For these volatile disqualification reasons, however, we must not try
     // to activate e10s because the backend doesn't know about it. E10S_STATUS
@@ -180,17 +181,17 @@ function defineCohort() {
     2: .5,
     4: .75,
     8: 1
   };
 
   let multiUserSample = getUserSample(true);
   for (let sampleName of Object.getOwnPropertyNames(BUCKETS)) {
     if (multiUserSample < BUCKETS[sampleName]) {
-      setCohort(`${cohortPrefix}multiBucket${sampleName}`);
+      setCohort(`multiBucket${sampleName}`);
       Preferences.set(PREF_E10S_PROCESSCOUNT + ".web", sampleName);
       break;
     }
   }
 }
 
 function shutdown(data, reason) {
 }