Closed
Bug 1515814
Opened 5 years ago
Closed 5 years ago
Fix SpecialPowers service worker cleanup functions for parent-intercept
Categories
(Core :: DOM: Service Workers, enhancement, P2)
Core
DOM: Service Workers
Tracking
()
RESOLVED
FIXED
mozilla66
Tracking | Status | |
---|---|---|
firefox66 | --- | fixed |
People
(Reporter: mrbkap, Assigned: mrbkap)
References
Details
Attachments
(2 files)
SpecialPowers has two functions to clear service workers from a test: removeAllServiceWorkerData and removeServiceWorkerDataForExampleDomain. They directly use an API that accesses the ServiceWorkerManager and that doesn't work in parent-intercept mode. I have a patch that makes SpecialPowers ask the parent to do the work for us.
Assignee | ||
Comment 1•5 years ago
|
||
These calls have to happen in the parent process to have any effect. I experimented with making ServiceWorkerCleanUp.jsm do the remoting itself but that didn't turn out to be worth the effort and it turns out all of the uses are currently in the parent process anyway.
Assignee | ||
Comment 2•5 years ago
|
||
This assures us that nobody misuses this code in the wrong process. Depends on D15160
Assignee | ||
Comment 3•5 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=313ec01abad67fb848a3597d79f85c34f1272e1
Updated•5 years ago
|
Priority: -- → P2
Pushed by mrbkap@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/64e898f29fd8 Proxy these calls to the parent. r=asuth https://hg.mozilla.org/integration/autoland/rev/bfb1e23872cc Ensure that nobody uses this in a child process. r=asuth
Comment 5•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/64e898f29fd8 https://hg.mozilla.org/mozilla-central/rev/bfb1e23872cc
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in
before you can comment on or make changes to this bug.
Description
•