Response constructor doesn't throw on disturbed ReadableStream

• Component: WebKit Misc.
• Status: RESOLVED FIXED

[Paul Rumkin]

It looks like a regression of the bug #156911

Example code which doesn't throw:

    try {
      const stream = new ReadableStream()
      stream.cancel();
    
      new Response(stream)
    
      console.log("Doesn't throw")
    }
    catch (_) {
      console.log("Throws")
    }

Expected behavior: Output "Throws"
Current behavior: Output "Doesn't throw"

Check online: https://jsbin.com/yitaloburu/edit?js,console

Safari version is 12.1.1

[Radar WebKit Bug Importer]

<rdar://problem/53550351>

[youenn fablet]

Created attachment 375515 [details]
Patch

[youenn fablet]

Created attachment 375516 [details]
Patch

[youenn fablet]

Created attachment 375521 [details]
Patch

[EWS Watchlist]

Comment on attachment 375521 [details]
Patch

Attachment 375521 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/12862664

New failing tests:
imported/w3c/web-platform-tests/fetch/api/request/request-init-stream.any.html

[EWS Watchlist]

Created attachment 375524 [details]
Archive of layout-test-results from ews103 for mac-highsierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-highsierra  Platform: Mac OS X 10.13.6

[EWS Watchlist]

Comment on attachment 375521 [details]
Patch

Attachment 375521 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/12862708

New failing tests:
imported/w3c/web-platform-tests/fetch/api/request/request-init-stream.any.html

[EWS Watchlist]

Created attachment 375525 [details]
Archive of layout-test-results from ews115 for mac-highsierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews115  Port: mac-highsierra  Platform: Mac OS X 10.13.6

[youenn fablet]

Created attachment 375552 [details]
Patch

[youenn fablet]

Created attachment 375565 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 375565 [details]
Patch

Clearing flags on attachment: 375565

Committed r248283: <https://trac.webkit.org/changeset/248283>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Sam Sneddon [:gsnedders]]

*** Bug 189042 has been marked as a duplicate of this bug. ***