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|.

Thanks for the discussion in person.
You are right.  We would eventually see the |fetchResponse| from queue because the result is put there regardless.
However, as we concluded it still lacks the way to reuse network response when both failed.  Let me revise this for that.

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

Message ID: <w3c/ServiceWorker/pull/1777/review/2916221175@github.com>