Closed
Bug 1240909
Opened 10 years ago
Closed 10 years ago
Fix test_idleapi_permissions.html for e10s
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla46
Tracking | Status | |
---|---|---|
firefox46 | --- | fixed |
People
(Reporter: mrbkap, Assigned: mrbkap)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
1.25 KB,
patch
|
mccr8
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•10 years ago
|
||
SpecialPowers.addPermission seems like it might be good grep fodder to find easy-to-fix mochitests.
Attachment #8709649 -
Flags: review?(continuation)
Comment 2•10 years ago
|
||
Comment on attachment 8709649 [details] [diff] [review]
Patch
Review of attachment 8709649 [details] [diff] [review]:
-----------------------------------------------------------------
I looked into that briefly (thought maybe for preferences?) and it seemed like there were a good number of tests that worked with e10s but still used the bad old version of the API. I guess if you don't use the pref/permission whatever quickly it works out okay or something? But certainly getting rid of the old version would be useful in any event.
Attachment #8709649 -
Flags: review?(continuation) → review+
Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/6ee3fa782ae4bf4cc5d446e06567c39f47b572e9
Bug 1240909 - Use pushPermissions to be e10s compatible. r=mccr8
Comment 4•10 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox46:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•