Re: [whatwg/streams] Refactor writable stream erroring to be centralized (#705)

ricea commented on this pull request.



> +          WritableStreamHandleAbortRequestIfPending(stream);
+        }
+
+        // This is a no-op if the stream was errored above.
+        WritableStreamDefaultControllerAdvanceQueueIfNeeded(this);
+      },
+      r => {
+        assert(stream._state === 'writable' || stream._state === 'errored');
+        WritableStreamDefaultControllerErrorIfNeeded(this, r);
+        WritableStreamRejectAbortRequestIfPending(stream);
+      }
+    )
+    .catch(rethrowAssertionErrorRejection);
+  }
+
+  [AbortSteps](reason) {

Should these be in alphabetical order?

-- 
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/705#pullrequestreview-29160032

Received on Monday, 27 March 2017 11:36:52 UTC