Closed Bug 1242775 Opened 8 years ago Closed 8 years ago

[e10s] Fix test_bug803225.html to work with e10s

Categories

(Core :: DOM: Security, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
e10s + ---
firefox47 --- affected
firefox48 --- fixed

People

(Reporter: mccr8, Assigned: mrbkap)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Once bug 1237726 is fixed, the remaining mixedcontentblocker mochitest that does not run with e10s is test_bug803225.html. The part of the test that does not work is in file_main_bug803225.html under "Test 7: mailto protocol". I think what the test is doing is starting up a link with the mailto: protocol, and checking whether or not it is actually loaded. The way this is being done does not work in a content process.
Blake, would you mind taking a look at this? I saw that you've done some work with e10s protocol handler stuff before. Thanks.
Flags: needinfo?(mrbkap)
I should have a fix for this tomorrow.
Assignee: nobody → mrbkap
Flags: needinfo?(mrbkap)
Thanks, Blake!
Attached patch Patch v1Splinter Review
The web handler app code has to run in the parent process. It opens a new window or tab which then loads in the child (in e10s) and fires the proper observer service obervers.
Attachment #8729219 - Flags: review?(continuation)
Attachment #8729219 - Flags: review?(continuation) → review+
https://hg.mozilla.org/mozilla-central/rev/47c57a3f7007
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: