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)

enhancement

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.
Blocks: 1462460
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.
This assures us that nobody misuses this code in the wrong process.

Depends on D15160
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
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.

Attachment

General

Created:
Updated:
Size: