[whatwg/streams] Don't error canceled branches when erroring a teed stream (#859)

The handler currently uses ReadableStreamDefaultControllerError, which runs ReadableStreamError on a closed stream, changing the state from "closed" to "errored" in the process.

Simply using ReadableStreamDefaultControllerErrorIfNeeded instead should be all that's needed.

Makes w3c/web-platform-tests#8299 pass.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Don't error canceled branches when erroring a teed stream

-- File Changes --

    M index.bs (4)
    M reference-implementation/lib/readable-stream.js (4)
    M reference-implementation/web-platform-tests (2)

-- Patch Links --

https://github.com/whatwg/streams/pull/859.patch
https://github.com/whatwg/streams/pull/859.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/859

Received on Friday, 17 November 2017 16:22:26 UTC