Closed Bug 1063156 Opened 5 years ago Closed 5 years ago

Enable e10s shims by default, even when browser.tabs.remote.autostart pref is false

Categories

(Core :: XPConnect, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla36
Tracking Status
e10s m4+ ---

People

(Reporter: cpeterson, Assigned: ally)

References

Details

Attachments

(1 file, 3 obsolete files)

The "File > Open e10s Window" menu item does not enable shims, so testers are reporting false positive bugs that are actually fixed when shims are enabled by browser.tabs.remote.autostart. I suggested that we just remove the "Open e10s Window" menu item in bug 1003313, but enabling shims all the time is better because shims will get more bake time (before we enable all of e10s) and casual testers can still try per-window e10s.

dom.compartment_per_addon (bug 1030420) to land first.
Moving old M2 P2 bugs to M4.
Status: NEW → ASSIGNED
 https://tbpl.mozilla.org/?tree=Try&rev=1c5cfe8ad4c9

let's see what breaks test-wise
Attached patch enableShimsByDefault (obsolete) — Splinter Review
try looks green to jimm & I. billm?
Attachment #8512087 - Flags: review?(wmccloskey)
Comment on attachment 8512087 [details] [diff] [review]
enableShimsByDefault

OK, let's do this. However, could you please make it nightly-only? The compartment-per-addon stuff is nightly-only, so this should be too.
Attachment #8512087 - Flags: review?(wmccloskey)
Attached patch enableShimsByDefault (obsolete) — Splinter Review
once more, with ifdefs!
Attachment #8512087 - Attachment is obsolete: true
Attachment #8512306 - Flags: review?(wmccloskey)
Attachment #8512306 - Flags: review?(wmccloskey) → review+
one of the perks of being a night owl today is that the trees are finally open!

  https://hg.mozilla.org/integration/fx-team/rev/8a1dfdaaeb3c
Not any more, I closed it because someone busted Android.

Backed out in https://hg.mozilla.org/integration/fx-team/rev/b0b2b78f2001 for making nearly every test suite on every flavor of Android time out in startup.
(In reply to Phil Ringnalda (:philor) from comment #7)
> Not any more, I closed it because someone busted Android.
> 
> Backed out in https://hg.mozilla.org/integration/fx-team/rev/b0b2b78f2001
> for making nearly every test suite on every flavor of Android time out in
> startup.

Well that didn't last long. Try was green on android too
okay, test ALL THE THINGS
remote: You can view the progress of your build at the following URL:
remote:   https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=c3ebccbfbc07
remote: Alternatively, view them on TBPL (soon to be deprecated):
remote:   https://tbpl.mozilla.org/?tree=Try&rev=c3ebccbfbc07
I wonder if Bug 1091707 is the cause. Try for android is either green or timed out, no failure summmary.
The cause of your backout? No, the problem there was that when automation.py went to start up Fennec, either it never started, or started but never produced whatever it is that automation.py looks for to know that it successfully started the app.

The cause of your try run not getting Android builds? No, that's some signing server bustage, which someone should file a bug in Release Engineering about, but instead everyone is just retriggering the builds until they get a successful one.
Backed out for that android bustage: 

https://hg.mozilla.org/integration/fx-team/rev/a31d8d3703f8
Flags: needinfo?(ally)
Attached patch enableShimsByDefault (obsolete) — Splinter Review
version designed to not break android. please note that preprocessor.py does not fully support the macro syntax, so the guards are different in cpp and js.
Attachment #8512306 - Attachment is obsolete: true
Flags: needinfo?(ally)
thanks Tom
Attachment #8515269 - Attachment is obsolete: true
remote:   https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=48272b5d82c6
remote: Alternatively, view them on TBPL (soon to be deprecated):
remote:   https://tbpl.mozilla.org/?tree=Try&rev=48272b5d82c6
https://hg.mozilla.org/mozilla-central/rev/c6d53703d449
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
Depends on: 1095067
Depends on: 1099416
Depends on: 1101182
Depends on: 1260945
You need to log in before you can comment on or make changes to this bug.