New issue
Advanced search Search tips

Issue 921096 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Jan 16
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

rtc_peer_connection.idl lists the same method twice.

Project Member Reported by jmedley@google.com, Jan 11

Issue description

Components: Blink>Bindings
Components: -Blink>Bindings Blink>WebRTC>PeerConnection
It's not strictly a duplicate; it's another overload. Whether or not it's intended for us to have both a single-argument and a 2-3-argument overload isn't a bindings issue -- it's a question for the WebRTC folks.

Comment 3 by guidou@chromium.org, Jan 16 (6 days ago)

Status: WontFix (was: Untriaged)
They are not the same method.
The first one is the modern promise-based version and the second one is a legacy callback-based version.
The callback-based version returns a Promise<void> because it is not allowed to have overloads that return different types. The callback-based version always returns a resolved promise that should be ignored.
See https://w3c.github.io/webrtc-pc/#dom-rtcpeerconnection-setlocaldescription
and https://w3c.github.io/webrtc-pc/#dom-rtcpeerconnection-setlocaldescription!overload-1 for the difference.

Sign in to add a comment