Bug 1359495 - Don't put disqualified users in multi buckets to fix cohort naming confusion. r=Felipe, a=gchang
authorBlake Kaplan <mrbkap@gmail.com>
Tue, 25 Apr 2017 13:54:40 -0700
changeset 396068 f825d935914655487138fb5934ed10d933ae0da0
parent 396067 67059080981c3de0a35a65b360ae833ddc30df50
child 396069 44acbe6acc81f1fac6d52aa94bc1aec9ef1978cd
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
bugs1359495
milestone54.0
Bug 1359495 - Don't put disqualified users in multi buckets to fix cohort naming confusion. r=Felipe, a=gchang
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 = {