Editorial: assert that async iterator's reader is always active

Fixes #1246.

Author: MattiasBuelens

index.bs

@@ -1012,8 +1012,7 @@ default-reader-asynciterator-prototype-internal-slots">Asynchronous iteration</h
  ignore>stream</var> and |iterator|, are:
 
  1. Let |reader| be |iterator|'s [=ReadableStream async iterator/reader=].
- 1. If |reader|.[=ReadableStreamGenericReader/[[stream]]=] is undefined, return [=a promise rejected
-    with=] a {{TypeError}}.
+ 1. Assert: |reader|.[=ReadableStreamGenericReader/[[stream]]=] is not undefined.
  1. Let |promise| be [=a new promise=].
  1. Let |readRequest| be a new [=read request=] with the following [=struct/items=]:
   : [=read request/chunk steps=], given |chunk|
@@ -1036,8 +1035,7 @@ default-reader-asynciterator-prototype-internal-slots">Asynchronous iteration</h
  ignore>stream</var>, |iterator|, and |arg|, are:
 
  1. Let |reader| be |iterator|'s [=ReadableStream async iterator/reader=].
- 1. If |reader|.[=ReadableStreamGenericReader/[[stream]]=] is undefined, return [=a promise resolved
-    with=] undefined.
+ 1. Assert: |reader|.[=ReadableStreamGenericReader/[[stream]]=] is not undefined.
  1. Assert: |reader|.[=ReadableStreamDefaultReader/[[readRequests]]=] is [=list/is empty|empty=],
     as the async iterator machinery guarantees that any previous calls to `next()` have settled
     before this is called.

reference-implementation/lib/ReadableStream-impl.js

@@ -119,11 +119,7 @@ exports.implementation = class ReadableStreamImpl {
 
   [idlUtils.asyncIteratorNext](iterator) {
     const reader = iterator._reader;
-    if (reader._stream === undefined) {
-      return promiseRejectedWith(
-        new TypeError('Cannot get the next iteration result once the reader has been released')
-      );
-    }
+    assert(reader._stream !== undefined);
 
     const promise = newPromise();
     const readRequest = {
@@ -143,10 +139,7 @@ exports.implementation = class ReadableStreamImpl {
 
   [idlUtils.asyncIteratorReturn](iterator, arg) {
     const reader = iterator._reader;
-    if (reader._stream === undefined) {
-      return promiseResolvedWith(undefined);
-    }
-
+    assert(reader._stream !== undefined);
     assert(reader._readRequests.length === 0);
 
     if (iterator._preventCancel === false) {