[slightlyoff/ServiceWorker] Handle Fetch creates JavaScript objects too early (and other feedback) (#951)

1. JavaScript objects are being created in step 4 (you also need to explicitly create a `Headers` object) without a global object. You should first find the service worker, start it if it hasn't started, then queue a task, and from that task create these objects.
2. Don't grab the headers attribute directly. Just use the associated concept.
3. In step 9 "corresponds" is not defined. E.g., what if request's client is null?
4. Also, if service worker client is just another word for environment settings object, why even mention it in step 9? Why not just set client to request's client?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/slightlyoff/ServiceWorker/issues/951

Received on Friday, 12 August 2016 13:02:01 UTC