If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Status

Testing Graveyard
Mozmill
RESOLVED FIXED
3 years ago
a year ago

People

(Reporter: Andrei Eftimie, Assigned: whimboo)

Tracking

unspecified
Bug Flags:
in-testsuite -

Details

(Whiteboard: [mozmill-2.0.9+])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

3 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.

Preference
----------
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.
(Assignee)

Comment 1

3 years ago
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.
(Assignee)

Updated

3 years ago
Whiteboard: [mozmill-2.1?] → [dupe bug 1065436?]
(Assignee)

Comment 2

3 years ago
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
Status: NEW → ASSIGNED
Whiteboard: [dupe bug 1065436?] → [mozmill-2.0.9+]
(Assignee)

Comment 3

3 years ago
(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!
(Assignee)

Updated

3 years ago
Blocks: 1082077
(Assignee)

Comment 4

3 years ago
Created attachment 8504196 [details] [diff] [review]
Finally disable v1

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)
(Assignee)

Comment 5

3 years ago
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)

Comment 6

3 years ago
You might want to take a look at this code to be sure everything is covered - 

http://mxr.mozilla.org/mozilla-central/source/toolkit/xre/nsAppRunner.cpp#4582

I wonder about that trial pref 'browser.tabs.remote.autostart.1' and subsequent flags.
(Assignee)

Comment 7

3 years ago
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.

Comment 8

3 years ago
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:

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

browser.tabs.remote.autostart.X
-------------------------------
not used yet, but will be used when we want to force a period of e10s on nightly builds for a few days

browser.displayedE10SPrompt.X
-----------------------------
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)
(Assignee)

Comment 10

3 years ago
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.
(Assignee)

Comment 12

3 years ago
Created attachment 8504374 [details] [diff] [review]
Finally disable v1.1

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+
(Reporter)

Comment 13

3 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+

Updated

3 years ago
tracking-e10s: --- → +
(Assignee)

Comment 14

3 years ago
https://github.com/mozilla/mozmill/commit/0d72d6e5195baed07636b907b557c82d05854bb4 (master)
https://github.com/mozilla/mozmill/commit/c2012e6a5cf817e4fb6a6a1675119e69244ded8d (hotfix-2.0)
Status: ASSIGNED → RESOLVED
Last Resolved: 3 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.