Closed Bug 1226708 Opened 9 years ago Closed 9 years ago

Enable most requestsync tests with e10s

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
e10s + ---
firefox45 --- fixed

People

(Reporter: mccr8, Assigned: mccr8)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

With bug 1226291, it is easy to get these tests to work: you just have to ensure that RequestSyncService.jsm is loaded in the main process. test_basic_app.html remains disabled because it still fails, but I assume it is just failing like all mozapps tests fail with e10s.
This makes them work with e10s.

The importInMainProcess patch got backed out for Android failures, so
this will have to wait until that can land, but otherwise this looked
fine on Try.

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6e209567bc24
Attachment #8690970 - Flags: review?(amarchesini)
They still do not pass, but now they fail instead of just timing out.
Attachment #8690971 - Flags: review?(amarchesini)
Attachment #8690970 - Flags: review?(amarchesini) → review+
Attachment #8690971 - Flags: review?(amarchesini) → review+
Keywords: checkin-needed
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: