Re: [slightlyoff/ServiceWorker] Create a new ReadableStream in FetchEvent.respondWith (#934) from Jungkee Song on 2016-08-03 (public-webapps-github@w3.org from August 2016)

From: Jungkee Song <notifications@github.com>
Date: Tue, 02 Aug 2016 20:08:53 -0700
To: slightlyoff/ServiceWorker <ServiceWorker@noreply.github.com>
Message-ID: <slightlyoff/ServiceWorker/pull/934/c237122426@github.com>

LGTM with the above nits and a rebase: we removed ServiceWorkerContainer.onerror in https://github.com/slightlyoff/ServiceWorker/commit/c240d9d57af1a4c5e2c0ff509e3136cba3fe649a. This patch is missing in this PR. Also, should I wait until your other PR to fetch gets landed? Or just merge this PR first?

---
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/pull/934#issuecomment-237122426

Received on Wednesday, 3 August 2016 03:09:21 UTC