Bug 1357959 - Put people who forced into e10s into the multi test group. r=Felipe

MozReview-Commit-ID: 3eAkCW8MtRf
This commit is contained in:
Blake Kaplan
2017-04-20 17:35:43 -07:00
parent f401c604cd
commit 494cc19377

View File

@@ -127,6 +127,7 @@ function defineCohort() {
setCohort("optedOut"); setCohort("optedOut");
} else if (userOptedIn.e10s) { } else if (userOptedIn.e10s) {
setCohort("optedIn"); setCohort("optedIn");
inMultiExperiment = true;
} else if (temporaryDisqualification != "") { } else if (temporaryDisqualification != "") {
// Users who are disqualified by the backend (from multiprocessBlockPolicy) // Users who are disqualified by the backend (from multiprocessBlockPolicy)
// can be put into either the test or control groups, because e10s will // can be put into either the test or control groups, because e10s will
@@ -185,7 +186,7 @@ function defineCohort() {
let multiUserSample = getUserSample(true); let multiUserSample = getUserSample(true);
for (let sampleName of Object.getOwnPropertyNames(BUCKETS)) { for (let sampleName of Object.getOwnPropertyNames(BUCKETS)) {
if (multiUserSample < BUCKETS[sampleName]) { if (multiUserSample < BUCKETS[sampleName]) {
setCohort(`${cohortPrefix}multiBucket${sampleName}`); setCohort(`multiBucket${sampleName}`);
Preferences.set(PREF_E10S_PROCESSCOUNT + ".web", sampleName); Preferences.set(PREF_E10S_PROCESSCOUNT + ".web", sampleName);
break; break;
} }