Closed
Bug 1237141
Opened 10 years ago
Closed 10 years ago
Make test_app_principal_equality.html work in e10s
Categories
(Core :: Security, defect)
Core
Security
Tracking
()
RESOLVED
FIXED
mozilla46
People
(Reporter: mrbkap, Assigned: mrbkap)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
4.16 KB,
patch
|
Felipe
:
review+
|
Details | Diff | Splinter Review |
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.
| Assignee | ||
Comment 1•10 years ago
|
||
| Assignee | ||
Comment 2•10 years ago
|
||
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)
Updated•10 years ago
|
Blocks: e10s-tests
tracking-e10s:
--- → +
Updated•10 years ago
|
Attachment #8704442 -
Flags: review?(felipc) → review+
| Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/372554a0019f752e3e180356cc2964561859c59b
Bug 1237141 - Make this test pass in e10s. r=felipe
Comment 4•10 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox46:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in
before you can comment on or make changes to this bug.
Description
•