Re: [w3c/ServiceWorker] ServiceWorkerAutoPreload (PR #1756)

@yoshisatoyanagisawa commented on this pull request.



> @@ -3228,9 +3228,11 @@ spec: storage; urlPrefix: https://storage.spec.whatwg.org/
       1. Let |shouldSoftUpdate| be true if any of the following are true, and false otherwise:
           * |request| is a [=non-subresource request=].
           * |request| is a [=subresource request=] and |registration| is [=stale=].
+      1. Let |raceResponse| be null.

I guess |raceResponse| can go to the line before L3293 to make its lifetime shorter?
We do not need to share |raceResponse| between "race-network-and-fetch-event" and SWAutpPreload.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/w3c/ServiceWorker/pull/1756#pullrequestreview-2930785272
You are receiving this because you are subscribed to this thread.

Message ID: <w3c/ServiceWorker/pull/1756/review/2930785272@github.com>

Received on Monday, 16 June 2025 06:29:58 UTC