Bug 243409
| Summary: | REGRESSION(249632@main): wpt /service-workers/service-worker/claim-worker-fetch.https.html | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Sam Sneddon [:gsnedders] <gsnedders> |
| Component: | Service Workers | Assignee: | Chris Dumez <cdumez> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | cdumez, webkit-bug-importer, youennf |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| URL: | https://www.w3.org/TR/service-workers/#control-and-use-worker-client | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=239066 | ||
Sam Sneddon [:gsnedders]
See https://wpt.fyi/results/service-workers/service-worker/claim-worker-fetch.https.html?label=master&product=safari-15.6%20%2817613.3.9.1.5%29%5Bstable%5D&product=safari%5Bexperimental%5D&aligned&view=subtest
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/97919820>
Sam Sneddon [:gsnedders]
Specifically, given I forgot what I was gonna write in the description, the `fetch() in blob URL Worker should be intercepted after the client is claimed.` subtest has regressed.
Chris Dumez
I think the issue is that in SWServer::claim(), doRegistrationMatching() returns false for the blob URL worker client since we call doRegistrationMatching() with clientData.url which is a Blob URL.
Chris Dumez
Relevant spec text:
https://www.w3.org/TR/service-workers/#control-and-use-worker-client
Chris Dumez
Spec doesn't actually deal properly with a service worker claiming a client worker that is a Blob URL but there is a spec bug about it:
- https://github.com/w3c/ServiceWorker/issues/1554
Chris Dumez
Pull request: https://github.com/WebKit/WebKit/pull/3164
EWS
Committed 253345@main (497df5ae79f3): <https://commits.webkit.org/253345@main>
Reviewed commits have been landed. Closing PR #3164 and removing active labels.