Bug 243409 - REGRESSION(249632@main): wpt /service-workers/service-worker/claim-worker-fetch.https.html
Summary: REGRESSION(249632@main): wpt /service-workers/service-worker/claim-worker-fet...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Service Workers (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL: https://www.w3.org/TR/service-workers...
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-08-01 09:55 PDT by Sam Sneddon [:gsnedders]
Modified: 2022-08-11 12:13 PDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 Radar WebKit Bug Importer 2022-08-01 09:55:45 PDT 
<rdar://problem/97919820>
Comment 2 Sam Sneddon [:gsnedders] 2022-08-01 09:56:22 PDT 
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.
Comment 3 Chris Dumez 2022-08-09 11:47:30 PDT 
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.
Comment 4 Chris Dumez 2022-08-09 11:50:45 PDT 
Relevant spec text:
https://www.w3.org/TR/service-workers/#control-and-use-worker-client
Comment 5 Chris Dumez 2022-08-09 12:27:12 PDT 
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
Comment 6 Chris Dumez 2022-08-09 13:15:45 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/3164
Comment 7 EWS 2022-08-11 12:13:27 PDT 
Committed 253345@main (497df5ae79f3): <https://commits.webkit.org/253345@main>

Reviewed commits have been landed. Closing PR #3164 and removing active labels.