- From: Domenic Denicola <notifications@github.com>
- Date: Fri, 22 Sep 2017 04:14:33 -0700
- To: whatwg/streams <streams@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Friday, 22 September 2017 11:14:55 UTC
domenic commented on this pull request. > @@ -277,7 +240,11 @@ class TransformStreamDefaultController { throw defaultControllerBrandCheckException('error'); } - TransformStreamError(this._controlledTransformStream, reason); + if (this._controlledTransformStream._readable._state !== 'readable') { + throw new TypeError('TransformStream is not in a state that can be errored'); + } + + TransformStreamDefaultControllerError(this, reason); Oh, I see, in parallel to the ones for other controllers. Yeah, maybe keep it, in which case LGTM for the whole patch. -- 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/800#discussion_r140468490
Received on Friday, 22 September 2017 11:14:55 UTC