1814459 - Implement RTCError for SDP parse errors

Status: RESOLVED FIXED

(Core :: WebRTC: Signaling, task, P3)

Description

[Byron Campen [:bwc]]

This covers the initial steps of adding RTCError and RTCErrorDetailTypes, refactoring JsepSession to be able to throw RTCError (as well as a plain DOMException), and filling out RTCError.sdpLineNumber.

Comment 1

[Anshul Malik]

I'd like to give it a try.

Comment 2

[Byron Campen [:bwc]]

The interop 2026 test-cases: https://wpt.live/webrtc/RTCError.html?interop-2026

Comment 3

[Anshul Malik]

Attachment: Bug 1814459 - Part 1. Add RTCError and RTCErrorEvent WebIDL interfaces and base C++ implementation. r?bwc

Adds the RTCError (DOMException subclass) and RTCErrorEvent (Event subclass)
interfaces as a prerequisite for plumbing RTCError through SDP, DTLS, SCTP,
and hardware encoder failure paths.

RTCError.sdpLineNumber and JsepSession integration are follow-up in this bug.

Comment 4

[Anshul Malik]

Attachment: Bug 1814459 - Part 2. Wire RTCError into SDP parse error rejection path. r?bwc

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/33b22a373fdf
https://hg.mozilla.org/integration/autoland/rev/fdce8626e275
Part 1. Add RTCError and RTCErrorEvent WebIDL interfaces and base C++ implementation. r=bwc,webidl,saschanaz,smaug
https://github.com/mozilla-firefox/firefox/commit/423113553307
https://hg.mozilla.org/integration/autoland/rev/2c30cb9e11be
Part 2. Wire RTCError into SDP parse error rejection path. r=bwc,webidl,smaug

Comment 6

[rperta]

https://hg.mozilla.org/mozilla-central/rev/fdce8626e275
https://hg.mozilla.org/mozilla-central/rev/2c30cb9e11be

Comment 7

[Hamish Willee]

Was the omission of the dev-doc-needed keyword deliberate - i.e. is there a reason this should not be documented?

If it needs to be, please add that keyword because otherwise MDN doesn't get a trigger to do anything. Note that compatibility data got recorded already in https://github.com/mdn/browser-compat-data/pull/29352 - also I am assuming that this is shipped as well, since there is no obvious preference gating this.