Closed Bug 1237141 Opened 10 years ago Closed 10 years ago

Make test_app_principal_equality.html work in e10s

Categories

(Core :: Security, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla46
Tracking Status
e10s + ---
firefox46 --- fixed

People

(Reporter: mrbkap, Assigned: mrbkap)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

This test sets prefs and permissions without waiting for them to take effect. There is no problem that can't be solved by adding more promises.
Attached patch Patch v1 — — Splinter Review
Basically, everything here had to become even more async. Annoyingly, we can't even create the iframes until we have the privilege. This uses a pretty weird feature of promises, where returning a promise from a promise's then function causes us to wait on *that* promise too.
Attachment #8704442 - Flags: review?(felipc)
Blocks: e10s-tests
tracking-e10s: --- → +
Attachment #8704442 - Flags: review?(felipc) → review+
Assignee: nobody → mrbkap
Blocks: 1218958
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: