Re: [whatwg/streams] Rewrite to use Web IDL, and generally modernize (#1035)

@ricea commented on this pull request.



> -{{ReadableByteStreamController}}. Those classes define most of the stateful internal slots and abstract
-operations for how a stream's <a>internal queue</a> is managed and how it interfaces with its <a>underlying source</a>
-or <a>underlying byte source</a>.
-
-Each controller class defines two internal methods, which are called by the {{ReadableStream}} algorithms:
-
-<dl>
-  <dt><dfn abstract-op lt="[[CancelSteps]]">\[[CancelSteps]](<var>reason</var>)</dfn></dt>
-  <dd>The controller's steps that run in reaction to the stream being <a lt="cancel a readable stream">canceled</a>,
-  used to clean up the state stored in the controller and inform the <a>underlying source</a>.</dd>
-
-  <dt><dfn abstract-op lt="[[PullSteps]]">\[[PullSteps]]()</dfn></dt>
-  <dd>The controller's steps that run when a <a>default reader</a> is read from, used to pull from the controller any
-  queued <a>chunks</a>, or pull from the <a>underlying source</a> to get more chunks.</dd>
+  <p>By default, calling the async iterator's `return()` method will also [=cancel a readable
+  stream|cancel=] the stream. To prevent this, use the stream's `values()`method, passing true for

```suggestion
  stream|cancel=] the stream. To prevent this, use the stream's `values()` method, passing true for
```

-- 
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/1035#pullrequestreview-398645432