Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(311)

Side by Side Diff: third_party/WebKit/Source/modules/peerconnection/RTCConfiguration.idl

Issue 2055553003: Change the default rtcp mux policy from negotiate to require. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix test Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // https://w3c.github.io/webrtc-pc/#rtcicetransportpolicy-enum 5 // https://w3c.github.io/webrtc-pc/#rtcicetransportpolicy-enum
6 6
7 // TODO(foolip): The spec does not have the "none" value. 7 // TODO(foolip): The spec does not have the "none" value.
8 // https://crbug.com/659134 8 // https://crbug.com/659134
9 enum RTCIceTransportPolicy { 9 enum RTCIceTransportPolicy {
10 "none", 10 "none",
(...skipping 20 matching lines...) Expand all
31 31
32 dictionary RTCConfiguration { 32 dictionary RTCConfiguration {
33 sequence<RTCIceServer> iceServers; 33 sequence<RTCIceServer> iceServers;
34 // TODO(foolip): |iceTransportPolicy| default should be "all", but it is 34 // TODO(foolip): |iceTransportPolicy| default should be "all", but it is
35 // omitted to allow fallback to |iceTransports| if not specified. 35 // omitted to allow fallback to |iceTransports| if not specified.
36 RTCIceTransportPolicy iceTransportPolicy; 36 RTCIceTransportPolicy iceTransportPolicy;
37 // TODO(foolip): |iceTransports| is not in the spec. 37 // TODO(foolip): |iceTransports| is not in the spec.
38 // https://crbug.com/659131 38 // https://crbug.com/659131
39 RTCIceTransportPolicy iceTransports; 39 RTCIceTransportPolicy iceTransports;
40 RTCBundlePolicy bundlePolicy = "balanced"; 40 RTCBundlePolicy bundlePolicy = "balanced";
41 // TODO(foolip): |rtcpMuxPolicy| default should be "require". 41 RTCRtcpMuxPolicy rtcpMuxPolicy = "require";
42 RTCRtcpMuxPolicy rtcpMuxPolicy;
43 // TODO(foolip): DOMString peerIdentity; 42 // TODO(foolip): DOMString peerIdentity;
44 sequence<RTCCertificate> certificates; 43 sequence<RTCCertificate> certificates;
45 // TODO(foolip): unsigned short iceCandidatePoolSize = 0; 44 // TODO(foolip): unsigned short iceCandidatePoolSize = 0;
46 }; 45 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698