stream: fix respondWithNewView() errors when view.byteOffset != 0

lib/internal/webstreams/readablestream.js

@@ -2965,7 +2965,7 @@ function readableByteStreamControllerRespondWithNewView(controller, view) {
   if (byteOffset + bytesFilled !== viewByteOffset)
     throw new ERR_INVALID_ARG_VALUE.RangeError('view', view);
 
-  if (bytesFilled + viewByteOffset > byteLength)
+  if (bytesFilled + viewByteLength > byteLength)
     throw new ERR_INVALID_ARG_VALUE.RangeError('view', view);
 
   if (bufferByteLength !== viewBufferByteLength)

test/parallel/test-whatwg-readablestream.js

@@ -1657,3 +1657,33 @@ class Source {
   reader.read(new DataView(buffer))
     .then(common.mustCall());
 }
+
+{
+  const stream = new ReadableStream({
+    type: 'bytes',
+    autoAllocateChunkSize: 128,
+    pull: common.mustCall((controller) => {
+      const view = controller.byobRequest.view;
+      const dest = new Uint8Array(
+        view.buffer,
+        view.byteOffset,
+        view.byteLength
+      );
+      dest.fill(1);
+      controller.byobRequest.respondWithNewView(dest);
+    }),
+  });
+
+  const reader = stream.getReader({ mode: 'byob' });
+
+  const buffer = new ArrayBuffer(10);
+  const view = new Uint8Array(
+    buffer,
+    1,
+    3
+  );
+
+  reader.read(view).then(common.mustCall(({ value }) => {
+    assert.deepStrictEqual(value, new Uint8Array([1, 1, 1]));
+  }));
+}