Bug 1359495 - Don't put disqualified users in multi buckets to fix cohort naming confusion. r=Felipe draft
authorBlake Kaplan <mrbkap@gmail.com>
Tue, 25 Apr 2017 13:54:40 -0700
changeset 568182 3015ebd8b99f4118b1eb5459b53eb320a1ec40f3
parent 568181 2c497462f25efd4f06da8b398526241a8fafffc5
child 568183 d8b27f59e94be16a84f1f71020ec9a9c3bd9ce53
push id55783
push userbmo:mrbkap@mozilla.com
push dateTue, 25 Apr 2017 21:08:31 +0000
reviewersFelipe
bugs1359495
milestone55.0a1
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 = {