- From: Yoshisato Yanagisawa <notifications@github.com>
- Date: Wed, 11 Jun 2025 02:21:55 -0700
- To: w3c/ServiceWorker <ServiceWorker@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <w3c/ServiceWorker/pull/1777/review/2916331591@github.com>
@yoshisatoyanagisawa commented on this pull request. > @@ -3279,7 +3282,14 @@ spec: storage; urlPrefix: https://storage.spec.whatwg.org/ 1. If |fetchHandlerResponse| is not null and not a [=network error=], and |raceFetchController| is not null, [=fetch controller/abort=] |raceFetchController|. 1. Let |raceFetchHandlerResult| be a [=race result=] whose [=race result/routed response=] is |fetchHandlerResponse| and [=race result/used route=] is {{RouterSourceEnum/"fetch-event"}}. 1. [=queue/Enqueue=] |raceFetchHandlerResult| to |queue|. - 1. Wait until |queue| is not empty. + 1. Set |fetchHandlerCompleted| to true. + 1. Wait until any of the following are true: + 1. |queue| is not empty. + 1. Both |networkFetchCompleted| and |cacheLookupCompleted| are true. + 1. If |queue| is empty, then: + 1. If |raceResponse|'s [=race response/value=] is a [=/response=], return |raceResponse|'s [=race response/value=]. + 1. If |raceResponse|'s [=race response/value=] is a [=network error=], return a [=network error=]. + 1. Return null. I feel this is typo. Also, it looks not meet what we want to do here. Let me remove the line. -- Reply to this email directly or view it on GitHub: https://github.com/w3c/ServiceWorker/pull/1777#discussion_r2139642508 You are receiving this because you are subscribed to this thread. Message ID: <w3c/ServiceWorker/pull/1777/review/2916331591@github.com>
Received on Wednesday, 11 June 2025 09:21:59 UTC