Re: [w3c/ServiceWorker] Add FetchEvent.handled (#1397) (#1496)

tingshao commented on this pull request.



> @@ -2932,6 +2940,7 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
       1. Let |client| be |request|'s [=request/client=].
       1. Let |reservedClient| be |request|'s [=request/reserved client=].
       1. Let |preloadResponse| be a new [=promise=].
+      1. Let |eventHandled| be a new [=promise=].

Thanks. I changed to declare |eventHandled| to be null fist in top-level to make it able to be accessed in those last steps. Then set it to a new promise when |activeWorker| is available. Hope this could also cover your idea.

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