1734761 - [wpt-sync] Sync PR 31163 - ReadableStream: Fix respond() after enqueue() with multiple pending reads

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 31163 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/31163
Details from upstream follow.

Adam Rice <[e-mail]> wrote:

ReadableStream: Fix respond() after enqueue() with multiple pending reads

The previous fix to calling respond() after enqueue() did not work
correctly when there were multiple pending calls to read(). Fix it.

Fixed: 1255762,1257465
Change-Id: I3f2dad7c3970c37579ee955491f74bbc7d70e456
Reviewed-on: https://chromium-review.googlesource.com/3213786
WPT-Export-Revision: bd84973ce7dd81bffd8c9bf1539e710620f32cb1

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

PR 31163 applied with additional changes from upstream: 73ff424533dc6b1277ab978b3be72cafebf7299c

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=2c86923b5436c2ca393d7c619040aedf4fccab37

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 5 tests and 3 subtests

Status Summary

Firefox

OK : 4
FAIL : 12

Chrome

CRASH: 4

Safari

OK : 3
FAIL : 9
ERROR: 1

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/streams/readable-byte-streams/respond-after-enqueue.any.js: SKIP (Chrome: SKIP, Safari: SKIP)
/streams/readable-byte-streams/respond-after-enqueue.any.sharedworker.html
byobRequest.respond() after enqueue() should not crash: FAIL
byobRequest.respond() with cached byobRequest after enqueue() should not crash: FAIL
byobRequest.respond() after enqueue() with double read should not crash: FAIL
/streams/readable-byte-streams/respond-after-enqueue.any.html
byobRequest.respond() after enqueue() should not crash: FAIL (Safari: FAIL)
byobRequest.respond() with cached byobRequest after enqueue() should not crash: FAIL (Safari: FAIL)
byobRequest.respond() after enqueue() with double read should not crash: FAIL (Safari: FAIL)
/streams/readable-byte-streams/respond-after-enqueue.any.worker.html
byobRequest.respond() after enqueue() should not crash: FAIL (Safari: FAIL)
byobRequest.respond() with cached byobRequest after enqueue() should not crash: FAIL (Safari: FAIL)
byobRequest.respond() after enqueue() with double read should not crash: FAIL (Safari: FAIL)
/streams/readable-byte-streams/respond-after-enqueue.any.serviceworker.html
byobRequest.respond() after enqueue() should not crash: FAIL (Safari: FAIL)
byobRequest.respond() with cached byobRequest after enqueue() should not crash: FAIL (Safari: FAIL)
byobRequest.respond() after enqueue() with double read should not crash: FAIL (Safari: FAIL)

Tests Disabled in Gecko Infrastructure

/streams/readable-byte-streams/respond-after-enqueue.any.js: SKIP (Chrome: SKIP, Safari: SKIP)

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fd41b5456a75
[wpt PR 31163] - ReadableStream: Fix respond() after enqueue() with multiple pending reads, a=testonly

Comment 5

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/fd41b5456a75