1489623 - Spec change: throw SyntaxError on RTCIceServer with no urls

Status: RESOLVED FIXED

(Core :: WebRTC, enhancement, P2)

Description

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

From https://github.com/w3c/webrtc-pc/pull/1978.

Turns out we already throw InvalidAccessError, so this ends up just being a rename.

Comment 1

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

From https://github.com/w3c/webrtc-pc/pull/1978

Comment 2

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Attachment: Bug 1489623 - Throw SyntaxError on empty urls in iceServers and switch to real SyntaxError over DOMException, + update WPT. r?ng

Comment 3

[Nico Grunbaum [:ng, @chew:mozilla.org]]

Comment on attachment 9007849 [details]
Bug 1489623 - Throw SyntaxError on empty urls in iceServers and switch to real SyntaxError over DOMException, + update WPT. r?ng

Nico Grunbaum [:ng] has approved the revision.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/531a559d65bc
Throw SyntaxError on empty urls in iceServers and switch to real SyntaxError over DOMException, + update WPT. r=ng

Comment 5

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

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/12937 for changes under testing/web-platform/tests

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/531a559d65bc

Comment 7

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

Upstream PR merged