Add asserts to ReadableStreamDefaultController

These asserts are in ReadableByteStreamController, so they should be here too.
whatwg · Sep 25, 2016 · 4be6fd1 · 4be6fd1
1 parent 3fd1246
commit 4be6fd1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/index.bs b/index.bs
@@ -1446,6 +1446,8 @@ ReadableStreamDefaultController(<var>stream</var>, <var>underlyingSource</var>,
   1. Resolve _startResult_ as a promise:
     1. Upon fulfillment,
       1. Set _controller_.[[started]] to *true*.
+      1. Assert: _controller_.[[pulling]] is *false*.
+      1. Assert: _controller_.[[pullAgain]] is *false*.
       1. Perform ! ReadableStreamDefaultControllerCallPullIfNeeded(_controller_).
     1. Upon rejection with reason _r_,
       1. Perform ! ReadableStreamDefaultControllerErrorIfNeeded(_controller_, _r_).

diff --git a/reference-implementation/lib/readable-stream.js b/reference-implementation/lib/readable-stream.js
@@ -976,6 +976,10 @@ class ReadableStreamDefaultController {
     Promise.resolve(startResult).then(
       () => {
         controller._started = true;
+
+        assert(controller._pulling === false);
+        assert(controller._pullAgain === false);
+
         ReadableStreamDefaultControllerCallPullIfNeeded(controller);
       },
       r => {