stream: improve views validation on BYOBRequest

- This throws if the view is zero-length when there is an active reader
when using `ReadableStreamBYOBRequest.respondWithNewView()`.

- By doing that, we can get all tests passed in
`readable-byte-streams/bad-buffers-and-views.any.js`.

Signed-off-by: Daeyeon Jeong [email protected]
PR-URL: #44155
Refs: https://streams.spec.whatwg.org/#readable-byte-stream-controller-respond-with-new-view
Reviewed-By: Luigi Pinca <[email protected]>

Author: daeyeon

lib/internal/webstreams/readablestream.js

@@ -2516,6 +2516,15 @@ function readableByteStreamControllerRespondWithNewView(controller, view) {
   const viewBuffer = ArrayBufferViewGetBuffer(view);
   const viewBufferByteLength = ArrayBufferGetByteLength(viewBuffer);
 
+  if (stream[kState].state === 'closed') {
+    if (viewByteLength !== 0)
+      throw new ERR_INVALID_STATE.TypeError('View is not zero-length');
+  } else {
+    assert(stream[kState].state === 'readable');
+    if (viewByteLength === 0)
+      throw new ERR_INVALID_STATE.TypeError('View is zero-length');
+  }
+
   const {
     byteOffset,
     byteLength,

test/wpt/status/streams.json

@@ -4,13 +4,5 @@
   },
   "transferable/deserialize-error.window.js": {
     "skip": "Browser-specific test"
-  },
-  "readable-byte-streams/bad-buffers-and-views.any.js": {
-    "fail": {
-      "note": "TODO: implement detached ArrayBuffer support",
-      "expected": [
-        "ReadableStream with byte source: respondWithNewView() throws if the supplied view's buffer is zero-length (in the readable state)"
-      ]
-    }
   }
 }