Bug 1359495 - Don't put disqualified users in multi buckets to fix cohort naming confusion. r=Felipe
authorBlake Kaplan <mrbkap@gmail.com>
Tue, 25 Apr 2017 13:54:40 -0700
changeset 354824 271acc9d32e5673145fc54a66d65c0bd4dbf64e6
parent 354823 0ce21d05acafd4c584b3c608737089b517b2f76f
child 354825 d83907ea33abff72b189ad08d055494b6e7866e1
push id41477
push usermrbkap@mozilla.com
push dateTue, 25 Apr 2017 21:44:38 +0000
treeherderautoland@35e9b30a0497 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersFelipe
bugs1359495
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 1359495 - Don't put disqualified users in multi buckets to fix cohort naming confusion. r=Felipe
browser/extensions/e10srollout/bootstrap.js
--- a/browser/extensions/e10srollout/bootstrap.js
+++ b/browser/extensions/e10srollout/bootstrap.js
@@ -164,16 +164,17 @@ function defineCohort() {
   // - If the user already opted into multi, then their prefs are already set
   //   correctly, we're done.
   // - If the user has addons that disqualify them for multi, leave them with
   //   the default number of content processes (1 on beta) but still in the
   //   test cohort.
   if (updateChannel !== "beta" ||
       !inMultiExperiment ||
       userOptedIn.multi ||
+      disqualified ||
       getAddonsDisqualifyForMulti()) {
     Preferences.reset(PREF_E10S_PROCESSCOUNT + ".web");
     return;
   }
 
   // The user is in the multi experiment!
   // Decide how many content processes to use for this user.
   let BUCKETS = {