fix(ext/file): resolve unresolved Promise in Blob.stream

[marcosc90]

This PR fixes some crashing WPT tests due to an unresolved promise.

---

This could be a stream spec bug

When controller.close is called on a byob stream, there's no cleanup of pending readIntoRequests. The only cleanup of pending readIntoRequests happen when .byobRequest.respond(0) is called, it happens here:

deno/ext/web/06_streams.js

Line 2026 in 6ba245f

function readableByteStreamControllerRespondInClosedState(

which ends up calling readIntoRequest.closeSteps(chunk); in

deno/ext/web/06_streams.js

Line 2070 in 6ba245f

readableStreamFulfillReadIntoRequest(stream, filledView, done);

To reproduce:

async function byobRead() {
  const input = [new Uint8Array([8, 241, 48, 123, 151])];
  const stream = new ReadableStream({
    type: "bytes",
    async pull(controller) {
      if(input.length === 0) {
        controller.close();
        // controller.byobRequest.respond(0); // uncomment for fix 
        return 
      }
      controller.enqueue(input.shift())
    },
  });

  const reader = stream.getReader({ mode: 'byob' });
  const r1 = await reader.read(new Uint8Array(64));
  console.log(r1);
  const r2 = await reader.read(new Uint8Array(64));
  console.log(r2);
}

await byobRead();

Running the script triggers:

error: Top-level await promise never resolved

[marcosc90]

A few more fetch tests were enabled: response-consume-stream.any.html & response-consume-stream.any.worker.html

test Getting an error Response stream ... ok
test Getting a redirect Response stream ... ok
test Read empty text response's body as readableStream ... ok
test Read empty blob response's body as readableStream ... ok
test Read blob response's body as readableStream with mode=undefined ... ok
test Read text response's body as readableStream with mode=undefined ... ok
test Read URLSearchParams response's body as readableStream with mode=undefined ... ok
test Read array buffer response's body as readableStream with mode=undefined ... ok
test Read form data response's body as readableStream with mode=undefined ... ok
test Read blob response's body as readableStream with mode=byob ... ok
test Read text response's body as readableStream with mode=byob ... failed (expected)
test Read URLSearchParams response's body as readableStream with mode=byob ... failed (expected)
test Read array buffer response's body as readableStream with mode=byob ... failed (expected)
test Read form data response's body as readableStream with mode=byob ... ok

[bartlomieju]

LGTM, great fix. Thank you @marcosc90!