Closed Bug 1876450 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 44184 - WebSocketStream: Remove extra magic of abort() and cancel() reason

Categories

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

Product:
Core
Component:
Networking: WebSockets
Type:
task

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Adam Rice <ricea@chromium.org> wrote:

WebSocketStream: Remove extra magic of abort() and cancel() reason

Previously if you passed an object with "code" and optionally "reason"
properties to reader.cancel() or writer.abort() it would try to
interpret them as the code and reason for the WebSocket close frame.
This feature was excessively magical and didn't provide enough value to
justify its existence.

Remove it.

BUG=983030

Change-Id: I88e7b33bf57acd8860ced750a2a68760e800d280
Reviewed-on: https://chromium-review.googlesource.com/5232965
WPT-Export-Revision: 47b08cf3eb7348a311e579d73c193323869000d8

Component: web-platform-tests → Networking: WebSockets
Product: Testing → Core
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.