Re: [whatwg/streams] Reject writer.ready promise on errors, instead of fulfilling (#526)

ricea commented on this pull request.



> @@ -0,0 +1,74 @@
+"use strict";
+
+self.recordingReadableStream = (extras = {}, strategy) => {
+  let controllerToCopyOver;
+  const stream = new ReadableStream({
+    start(controller) {
+      controllerToCopyOver = controller;
+
+      if (extras.start) {

In Chrome and Blink style we usually leave out the {} if the if statement can be written on 2 lines.

-- 
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/526#pullrequestreview-3257912