Re: [streams] Byte stream update, including reference implementation (#361)

> +    if (stream._closeRequested) {
> +      throw new TypeError('The stream has already been closed; do not close it again!');
> +    }
> +    if (stream._state !== 'readable') {
> +      throw new TypeError('The stream is not in the readable state and cannot be closed');
> +    }
> +
> +    if (this._totalQueuedBytes > 0) {
> +      this._closeRequested = true;
> +
> +      return;
> +    }
> +
> +    const reader = stream._reader;
> +
> +    if (reader === undefined || IsReadableByteStreamReader(reader)) {

I think the naming of IsReadableByteStreamReader is a bit confusing, even though it mirrors the getReader/getByobReader methods. Maybe IsReadableByteStreamDefaultReader?

---
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/361/files#r33549270