[wpt-sync] Sync PR 21825 - WebSocketStream: Use a hwm of 1 for the readable
Categories
(Core :: Networking: WebSockets, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox75 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream][necko-triaged])
Sync web-platform-tests PR 21825 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/21825
Details from upstream follow.
Adam Rice <ricea@chromium.org> wrote:
WebSocketStream: Use a hwm of 1 for the readable
Initially WebSocketStream used a hwm of 0 for the ReadableStream that is
returned from theconnected
promise, but that resulted in poor
throughput when the application wasn't continuously reading.Change the hwm of 1 so that up to one message will still be transferred
to the render process even when it is not inside a read() call.
Backpressure will still be applied if more than 1 message is in transit.BUG=983030
Change-Id: I883d9e62b8465d45bc9b383db8f639fbf791c78d
Reviewed-on: https://chromium-review.googlesource.com/2054605
WPT-Export-Revision: 7fb1201a0212da05f948a1f637814fdc9c806281
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=6236bd500b9c01853154afba7454e56406c7a46c
Assignee | ||
Comment 2•4 years ago
|
||
CI Results
Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 4 tests and 1 subtests
Status Summary
Firefox
OK : 4
FAIL : 4
Chrome
OK : 4
PASS : 4
Safari
OK : 3
FAIL : 3
ERROR: 1
Links
Details
New Tests That Don't Pass
/websockets/stream-tentative/backpressure-receive.any.html
backpressure should be applied to received messages: FAIL (Chrome: PASS, Safari: FAIL)
/websockets/stream-tentative/backpressure-receive.any.worker.html
backpressure should be applied to received messages: FAIL (Chrome: PASS, Safari: FAIL)
/websockets/stream-tentative/backpressure-receive.any.serviceworker.html
backpressure should be applied to received messages: FAIL (Chrome: PASS, Safari: FAIL)
/websockets/stream-tentative/backpressure-receive.any.sharedworker.html
backpressure should be applied to received messages: FAIL (Chrome: PASS)
Updated•4 years ago
|
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d2b5be822478 [wpt PR 21825] - WebSocketStream: Use a hwm of 1 for the readable, a=testonly
Comment 4•4 years ago
|
||
bugherder |
Description
•