Chromium Code Reviews| Index: Source/modules/mediastream/RTCPeerConnection.cpp |
| diff --git a/Source/modules/mediastream/RTCPeerConnection.cpp b/Source/modules/mediastream/RTCPeerConnection.cpp |
| index 51a5d99241a3b761e296aeab803183121f94da90..e313c85c4d73f5dc4b5bc60823001e7debc34010 100644 |
| --- a/Source/modules/mediastream/RTCPeerConnection.cpp |
| +++ b/Source/modules/mediastream/RTCPeerConnection.cpp |
| @@ -129,9 +129,21 @@ RTCConfiguration* RTCPeerConnection::parseConfiguration(const Dictionary& config |
| } |
| } |
| + RTCRtcpMuxPolicy rtcpMuxPolicy = RTCRtcpMuxPolicyRequire; |
|
tommi (sloooow) - chröme
2015/05/22 15:46:10
Shouldn't this be initialized to WebRTCRtcpMuxPoli
pthatcher2
2015/05/22 15:55:27
Yes, thank you for noticing that. That could have
|
| + String rtcpMuxPolicyString; |
| + if (DictionaryHelper::get(configuration, "rtcpMuxPolicy", rtcpMuxPolicyString)) { |
| + if (rtcpMuxPolicyString == "require") { |
| + rtcpMuxPolicy = RTCRtcpMuxPolicyRequire; |
| + } else if (rtcpMuxPolicyString != "negotiate") { |
| + exceptionState.throwTypeError("Malformed RTCRtcpMuxPolicy"); |
| + return 0; |
| + } |
| + } |
| + |
| RTCConfiguration* rtcConfiguration = RTCConfiguration::create(); |
| rtcConfiguration->setIceTransports(iceTransports); |
| rtcConfiguration->setBundlePolicy(bundlePolicy); |
| + rtcConfiguration->setRtcpMuxPolicy(rtcpMuxPolicy); |
| for (size_t i = 0; i < numberOfServers; ++i) { |
| Dictionary iceServer; |