- 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