5 years ago
3 years ago


(Reporter: andrei, Assigned: whimboo)


Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags



(Whiteboard: [mozmill-2.0.9+])


(1 attachment, 1 obsolete attachment)



5 years ago
I've seen the e10s prompt interfere with our tests (l10n test noticeably)

We should disable it globally.
The following preference takes care of it.

Name: `browser.displayedE10SPrompt.1`
Value: `5`

I'd include this in the next mozmill release, we could try disabling this in our tests as a workaround if it disrupts more than right now.
This was already fixed in Mozmill with release 2.0.7 via bug 1065436. Please check what version of mozmill you are using. It looks like an outdated one.
Whiteboard: [mozmill-2.1?] → [dupe bug 1065436?]
Turned out we use a wrong pref in Mozmill, which needs a fix. Waiting for the reply from Jimm on bug 1065436 before we can get started here. But yeah, looks like we need a 2.0.9 release of Mozmill.
Assignee: nobody → hskupin
Whiteboard: [dupe bug 1065436?] → [mozmill-2.0.9+]
(In reply to Jim Mathies [:jimm] from bug 1065436 comment #8)
> > +            # Bug 695026 - Re-enable e10s when fully supported
> > +            preferences.append(('browser.tabs.remote', False))
> > +            preferences.append(('browser.tabs.autostart', False))
> This should be 'browser.tabs.remote.autostart'. Also 'browser.tabs.remote'
> was depreciated about a week ago.

After talking to Jim, I will remove the remote pref, and update the autostart. Thanks for noticing both of you!
Posted patch Finally disable v1 (obsolete) — Splinter Review
Strangely I'm not able to reproduce this problem at all on my Linux box. Whether with Mozmill directly nor the automation scripts. So we will have to wait for Andrei to verify that it completely fixes the problem.
Attachment #8504196 - Flags: review?(ahalberstadt)
Attachment #8504196 - Flags: feedback?(andrei.eftimie)
Comment on attachment 8504196 [details] [diff] [review]
Finally disable v1

Review of attachment 8504196 [details] [diff] [review]:

Today is Canadian holiday. So lets move the review request over to Chris.
Attachment #8504196 - Flags: review?(ahalberstadt) → review?(cmanchester)
You might want to take a look at this code to be sure everything is covered -

I wonder about that trial pref 'browser.tabs.remote.autostart.1' and subsequent flags.
Would you mind telling for what this additional pref is used? All the flags for e10s mode are way too confusing. Things have changed a lot, and I currently do not have the time to dive into all that Firefox code. I would appreciate to know if we have to also add this trial pref to Mozmill.
lets ask felipe, he knows more about that .1 pref than I do.
Flags: needinfo?(felipc)
The prefs are confusing but we've been simplifying them. Basically there are 3 prefs now:

meant for users to manually toggle (through the "enable e10s" checkbox Preferences dialog) when they want to opt-in to e10s testing

not used yet, but will be used when we want to force a period of e10s on nightly builds for a few days

will be used (and X incremented) each time we have a period where we want to offer the e10s prompt asking users to opt-in

So, in order to both disable the prompt, and e10s itself, and avoid having to deal with the same bug in the future, I suggest that you set all of:

browser.tabs.remote.autostart = false

browser.tabs.remote.autostart.1 = false
browser.tabs.remote.autostart.2 = false
browser.tabs.remote.autostart.3 = false
browser.tabs.remote.autostart.4 = false

browser.displayedE10SPrompt.1 = 5
browser.displayedE10SPrompt.2 = 5
browser.displayedE10SPrompt.3 = 5
browser.displayedE10SPrompt.4 = 5

And then remove these when bug 695026 gets fixed.
Flags: needinfo?(felipc)
Thanks Felipe. Can we be sure we only have indices from 1 to 4, or should we account more?
Not sure if there will be more or not, I guess it depends on how the experiments with Nightly users go. I guess you can add more as it's trivial to do so, but too much would be unnecessary. I'll keep an eye and if we ever end up going higher again I can ping you or open a new bug to add more.
Ok, this adds the additional prefs and even some more (up to .9). That should be enough and we may not have to release another 2.0.x release of Mozmill.
Attachment #8504196 - Attachment is obsolete: true
Attachment #8504196 - Flags: review?(cmanchester)
Attachment #8504196 - Flags: feedback?(andrei.eftimie)
Attachment #8504374 - Flags: review?(cmanchester)
Attachment #8504374 - Flags: feedback?(felipc)
Attachment #8504374 - Flags: feedback?(andrei.eftimie)
Attachment #8504374 - Flags: feedback?(felipc) → feedback+
Attachment #8504374 - Flags: review?(cmanchester) → review+

Comment 13

5 years ago
Comment on attachment 8504374 [details] [diff] [review]
Finally disable v1.1

Review of attachment 8504374 [details] [diff] [review]:

Works fine.
Attachment #8504374 - Flags: feedback?(andrei.eftimie) → feedback+


5 years ago
tracking-e10s: --- → + (master) (hotfix-2.0)
Last Resolved: 5 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.