Re: [w3c/ServiceWorker] Isolate fix for race-network-and-fetch-handler stall from PR #1764 (PR #1777)

@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