[whatwg/streams] fix autoAllocateChunkSize (#651)

It was special-casing the first read request, and adding a second one would assert ever since it was first merged in https://github.com/whatwg/streams/pull/430#issuecomment-195370054.

```js
var rs = new ReadableStream({type:'bytes', autoAllocateChunkSize:1});
var r = rs.getReader();
var p1 = r.read();
var p2 = r.read(); // assertion failure
```

Also combined two redundant nested if statements, since `totalQueuedBytes > 0` implies `ReadableStreamGetNumReadRequests(stream) === 0`, AFAICT.
You can view, comment on, or merge this pull request online at:

  https://github.com/whatwg/streams/pull/651

-- Commit Summary --

  * fix autoAllocateChunkSize
  * fixup whitespace indent

-- File Changes --

    M index.bs (38)
    M reference-implementation/lib/readable-stream.js (64)

-- Patch Links --

https://github.com/whatwg/streams/pull/651.patch
https://github.com/whatwg/streams/pull/651.diff

-- 
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/pull/651

Received on Saturday, 14 January 2017 01:49:27 UTC