1357959 - Fix cohort name edge cases in the beta 54 multi experiement

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Blake Kaplan (:mrbkap) (inactive)]

The most important problem is that an extension that blocks multi but not e10s (e.g. a bootstrapped extension) will show up as it did for the e10s single experiment, which won't be super clear.

Comment 1

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Fix cohort names in edge cases for the beta experiment

There are a few cases where the cohort names aren't as clear as they could be.
In those cases, we can recuperate the data by looking at a combination of the
prefs that were set, but it would be nice to fix them up.

This also fixes a bug where if the user explicitly opted into e10s (that is,
they set browser.tabs.remote.force-enable) they wouldn't be placed in the
multi experiment.

MozReview-Commit-ID: 5wpfQDvbJJ3

Comment 2

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Updated patch

After talking to Felipe on IRC, we decided to tweak a couple of things here. In particular, the suffix -test in the cohort will mean "eligible for e10s but disqualified by an addon from multi." -control will mean "selected to not be in e10s at all" and to reduce confusion, we're not going to bother with the cohort prefix for the multi buckets since we don't have different addon policies for multi.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4bc74de54b5b
Put people who forced into e10s into the multi test group. r=Felipe

Comment 4

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 8860202 [details] [diff] [review]
Updated patch

Approval Request Comment
[Feature/Bug causing the regression]: bug 1349363
[User impact if declined]: Some users will not be included in the e10s-multi experiment.
[Is this code covered by automated tests?]: No :(
[Has the fix been verified in Nightly?]: No.
[Needs manual test from QE? If yes, steps to reproduce]: See bug 1352388
[List of other uplifts needed for the feature/fix]: n/a
[Is the change risky?]: No.
[Why is the change risky/not risky?]: This simply changes a couple of strings and sets a variable that should have been set.
[String changes made/needed]: n/a

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/4bc74de54b5b

Comment 6

[Gerry Chang [:gchang]]

Comment on attachment 8860202 [details] [diff] [review]
Updated patch

This patch is for e10s-multi experiment. Beta54+. Should be in 54 beta 2.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/7f717abab712