Closed Bug 1222495 Opened 6 years ago Closed 6 years ago
_suspend .html working with e10s
This test fails because it sets preferences directly using the preference service rather than using SpecialPowers.pushPrefEnv(). This seems to flip the pref on and off a number of times, and has a bunch of interconnected callbacks. The control flow will probably need to be rewritten to use Promises or something but I haven't been able to get my head around it yet.
I misunderstood how this works. pushPrefEnv still uses a matching popPrefEnv. So it shouldn't be too bad.
Assignee: nobody → continuation
This is the last plain Mochitest in dom/workers/test/ that is not enabled with e10s. try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=0258ece2c04d
Attachment #8686089 - Flags: review?(khuey)
I wonder if this tests passes on b2g with these changes. Is it worth pushing to try with it enabled there?
(In reply to Blake Kaplan (:mrbkap) (please use needinfo!) from comment #3) > I wonder if this tests passes on b2g with these changes. Is it worth pushing > to try with it enabled there? I don't know of any reason it wouldn't. I'll do a try push.
Attachment #8686089 - Flags: review?(khuey) → review+
B2G try run looked fine so I'm enabling it there, too: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6148567df12c
You need to log in before you can comment on or make changes to this bug.