Closed Bug 1643722 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 24011 - [Reland] [WebSocket] Test header + close frame in same packet

Categories

(Core :: Networking: WebSockets, task, P4)

task

Tracking

()

RESOLVED FIXED
mozilla79
Tracking Status
firefox79 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Adam Rice <ricea@chromium.org> wrote:

[Reland] [WebSocket] Test header + close frame in same packet

Modify the existing web platform test opening-handshake/005.html to send
the response header and the close frame in the same packet. This
verifies that Chrome passes the close data to JavaScript correctly in
this case.

This increases the coverage of the existing test, so there's no need to
retain the old behaviour.

Also remove the timeout expectations for this test, and make sure it
doesn't timeout on failure.

Originally landed as
https://chromium-review.googlesource.com/c/chromium/src/+/2228252. This
reland contains a fix for a flakiness problem with the previous CL.

Closed: 1090210
Change-Id: I89222d78f78a8251043fffe5ed860bc402882a9e
Reviewed-on: https://chromium-review.googlesource.com/2231923
WPT-Export-Revision: bdbe439c77bf64c644ae63e450a09babd23fa2f3

Component: web-platform-tests → Networking: WebSockets
Product: Testing → Core

CI Results

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

Total 2 tests and 1 subtests

Status Summary

Firefox

OK : 2
PASS: 2

Chrome

OK : 2
PASS: 2

Safari

OK : 2
PASS: 2

Links

GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c38d8af09f5f
[wpt PR 24011] - [Reland] [WebSocket] Test header + close frame in same packet, a=testonly
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79
You need to log in before you can comment on or make changes to this bug.