Re: [whatwg/streams] Change default readableStrategy HWM to 0? (#777)

I like your suggestion better so that is what I am working on.

I have run into an interesting problem with this test:

```javascript
promise_test(() => {
  const ts = new TransformStream();

  const writer = ts.writable.getWriter();
  writer.close();

  return Promise.all([writer.closed, ts.readable.getReader().closed]);
}, 'TransformStream: by default, closing the writable closes the readable (when there are no queued writes)');
```

This test never completes with HWM 0 because the promise returned by TransformWriterDefaultSink's start() method never resolves. I think this is pretty clearly bad behaviour.

Strictly speaking, this has always been broken, but making the readable HWM default to 0 makes it a lot more obvious.

I am looking at revising the state machine so that we don't need to delay returning from start().

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/issues/777#issuecomment-326574369

Received on Friday, 1 September 2017 13:03:21 UTC