Re: [ServiceWorker] Backpressure on fetch integrated with Streams (#452)

> I'd like to confirm if your opinion is the same as Anne regarding the following question:
>
> fetch(req), new Request(req) and cache.put(req, res) fails if any of fetch(req), new Request(req) and cache.put(req, res) has happened, even after the operation is finished.

I agree with this.

> Domenic, which is depicting your idea?

(A)'' depicts my idea.

> BTW, Yutaka questioned what we should do with the ReadableStream when e.g. .text() fails due to invalid UTF-8 data. We came up with the following options:

I think 1 or 2 are reasonable and I don't have a strong preference between them. 1 would be text() doing nothing, and 2 would be text() calling `this.body.cancel(new TypeError("couldn't decode chunk!"))`. Maybe since we already think of text() as saying "I want all of the remaining data", 2 is best.



---
Reply to this email directly or view it on GitHub:
https://github.com/slightlyoff/ServiceWorker/issues/452#issuecomment-72267929

Received on Friday, 30 January 2015 20:49:54 UTC