New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 659035 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Change RTCConfiguration's rtcpMuxPolicy default to "require" (is "negotiate")

Project Member Reported by foolip@chromium.org, Oct 25 2016

Issue description

Found as part of  issue 658423  and https://codereview.chromium.org/2442763002

The WebRTC.PeerConnection.SelectedRtcpMuxPolicy and WebRTC.PeerConnection.RtcpMux histograms are relevant.
 
Components: Blink>WebRTC>PeerConnection
Components: -Blink>WebRTC>Network
Changed label to PeerConnection, since this is more a Blink issue than an ICE protocol issue.

Comment 3 by hta@webrtc.org, Nov 11 2016

@foolip clarification - when the RTCConfiguration argument is optional, and the default for rtcpMuxPolicy is "require", that means that the implementation has to behave as if "require" was specified, right?

Otherwise, "new RTCPeerConnection()" and "new RTCPeerConnection({})" would have different behavior, and that would be strange.

So an implementation can't have a different default than the spec.

Comment 4 by foolip@chromium.org, Nov 11 2016

Right, all of these need to do the same thing:
new RTCPeerConnection()
new RTCPeerConnection(null)
new RTCPeerConnection(undefined)
new RTCPeerConnection({})
new RTCPeerConnection({ rtcpMuxPolicy: undefined })

Right now, they all act like new RTCPeerConnection({ rtcpMuxPolicy: "negotiate" })
FYI, here's a related bug for changing the default in the native library: https://bugs.chromium.org/p/webrtc/issues/detail?id=6030

Comment 6 by guidou@chromium.org, Nov 14 2016

Owner: hbos@chromium.org
Status: Assigned (was: Untriaged)
Status: Verified (was: Assigned)

Sign in to add a comment