Change the default rtcp mux policy from negotiate to require.

third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 211 matching lines @@
   if (policy == "none")
     return WebRTCIceTransportPolicy::kNone;
   if (policy == "relay")
     return WebRTCIceTransportPolicy::kRelay;
   DCHECK_EQ(policy, "all");
   return WebRTCIceTransportPolicy::kAll;
 }
 
 WebRTCConfiguration parseConfiguration(ExecutionContext* context,
                                        const RTCConfiguration& configuration,
-                                       ExceptionState& exceptionState,
-                                       RtcpMuxPolicy* selectedRtcpMuxPolicy) {
+                                       ExceptionState& exceptionState) {
   DCHECK(context);
-  DCHECK(selectedRtcpMuxPolicy);
 
   WebRTCIceTransportPolicy iceTransportPolicy = WebRTCIceTransportPolicy::kAll;
   if (configuration.hasIceTransportPolicy()) {
     UseCounter::count(context, UseCounter::RTCConfigurationIceTransportPolicy);
     iceTransportPolicy =
         iceTransportPolicyFromString(configuration.iceTransportPolicy());
     if (iceTransportPolicy == WebRTCIceTransportPolicy::kNone) {
       UseCounter::count(context,
                         UseCounter::RTCConfigurationIceTransportPolicyNone);
     }
   } else if (configuration.hasIceTransports()) {
     UseCounter::count(context, UseCounter::RTCConfigurationIceTransports);
     iceTransportPolicy =
         iceTransportPolicyFromString(configuration.iceTransports());
     if (iceTransportPolicy == WebRTCIceTransportPolicy::kNone)
       UseCounter::count(context, UseCounter::RTCConfigurationIceTransportsNone);
   }
 
   WebRTCBundlePolicy bundlePolicy = WebRTCBundlePolicy::kBalanced;
   String bundlePolicyString = configuration.bundlePolicy();
   if (bundlePolicyString == "max-compat") {
     bundlePolicy = WebRTCBundlePolicy::kMaxCompat;
   } else if (bundlePolicyString == "max-bundle") {
     bundlePolicy = WebRTCBundlePolicy::kMaxBundle;
   } else {
     DCHECK_EQ(bundlePolicyString, "balanced");
   }
 
-  // For the histogram value of "WebRTC.PeerConnection.SelectedRtcpMuxPolicy".
-  *selectedRtcpMuxPolicy = RtcpMuxPolicyDefault;
-  WebRTCRtcpMuxPolicy rtcpMuxPolicy = WebRTCRtcpMuxPolicy::kNegotiate;
+  WebRTCRtcpMuxPolicy rtcpMuxPolicy = WebRTCRtcpMuxPolicy::kRequire;
   if (configuration.hasRtcpMuxPolicy()) {

[foolip, 2016/12/14 19:24:47]

Now that rtcpMuxPolicy has a default value in the IDL, you can remove this
check.

[zhihuang1, 2016/12/15 19:14:28]

Done.

     String rtcpMuxPolicyString = configuration.rtcpMuxPolicy();
-    if (rtcpMuxPolicyString == "require") {
-      *selectedRtcpMuxPolicy = RtcpMuxPolicyRequire;
-      rtcpMuxPolicy = WebRTCRtcpMuxPolicy::kRequire;
-    } else {
+    if (rtcpMuxPolicyString != "require") {

[foolip, 2016/12/14 19:24:47]

To match the above structure, I'd do:

if (rtcpMuxPolicyString == "negotiate") {
  rtcpMuxPolicy = WebRTCRtcpMuxPolicy::kNegotiate;
} else {
  DCHECK_EQ(rtcpMuxPolicyString, "require");
}

[zhihuang1, 2016/12/15 19:14:28]

Done.

       DCHECK_EQ(rtcpMuxPolicyString, "negotiate");
-      *selectedRtcpMuxPolicy = RtcpMuxPolicyNegotiate;
+      rtcpMuxPolicy = WebRTCRtcpMuxPolicy::kNegotiate;
     }
   }
 
   WebRTCConfiguration webConfiguration;
   webConfiguration.iceTransportPolicy = iceTransportPolicy;
   webConfiguration.bundlePolicy = bundlePolicy;
   webConfiguration.rtcpMuxPolicy = rtcpMuxPolicy;
 
   if (configuration.hasIceServers()) {
     Vector<WebRTCIceServer> iceServers;
@@ skipping 149 matching lines @@
     const RTCConfiguration& rtcConfiguration,
     const Dictionary& mediaConstraints,
     ExceptionState& exceptionState) {
   if (mediaConstraints.isObject())
     UseCounter::count(context,
                       UseCounter::RTCPeerConnectionConstructorConstraints);
   else
     UseCounter::count(context,
                       UseCounter::RTCPeerConnectionConstructorCompliant);
 
-  // Record the RtcpMuxPolicy for histogram
-  // "WebRTC.PeerConnection.SelectedRtcpMuxPolicy".
-  RtcpMuxPolicy selectedRtcpMuxPolicy = RtcpMuxPolicyDefault;
-  WebRTCConfiguration configuration = parseConfiguration(
-      context, rtcConfiguration, exceptionState, &selectedRtcpMuxPolicy);
+  WebRTCConfiguration configuration =
+      parseConfiguration(context, rtcConfiguration, exceptionState);
   if (exceptionState.hadException())
     return 0;
 
   // Make sure no certificates have expired.
   if (configuration.certificates.size() > 0) {
     DOMTimeStamp now = convertSecondsToDOMTimeStamp(currentTime());
     for (const std::unique_ptr<WebRTCCertificate>& certificate :
          configuration.certificates) {
       DOMTimeStamp expires = certificate->expires();
       if (expires <= now) {
@@ skipping 11 matching lines @@
     mediaErrorState.raiseException(exceptionState);
     return 0;
   }
 
   RTCPeerConnection* peerConnection = new RTCPeerConnection(
       context, configuration, constraints, exceptionState);
   peerConnection->suspendIfNeeded();
   if (exceptionState.hadException())
     return 0;
 
-  peerConnection->m_peerHandler->logSelectedRtcpMuxPolicy(
-      selectedRtcpMuxPolicy);
-
   return peerConnection;
 }
 
 RTCPeerConnection::RTCPeerConnection(ExecutionContext* context,
                                      const WebRTCConfiguration& configuration,
                                      WebMediaConstraints constraints,
                                      ExceptionState& exceptionState)
     : ActiveScriptWrappable(this),
       ActiveDOMObject(context),
       m_signalingState(SignalingStateStable),
@@ skipping 313 matching lines @@
   return RTCSessionDescription::create(webSessionDescription);
 }
 
 void RTCPeerConnection::updateIce(ExecutionContext* context,
                                   const RTCConfiguration& rtcConfiguration,
                                   const Dictionary& mediaConstraints,
                                   ExceptionState& exceptionState) {
   if (throwExceptionIfSignalingStateClosed(m_signalingState, exceptionState))
     return;
 
-  RtcpMuxPolicy selectedRtcpMuxPolicy = RtcpMuxPolicyDefault;
-  WebRTCConfiguration configuration = parseConfiguration(
-      context, rtcConfiguration, exceptionState, &selectedRtcpMuxPolicy);
+  WebRTCConfiguration configuration =
+      parseConfiguration(context, rtcConfiguration, exceptionState);
 
   if (exceptionState.hadException())
     return;
 
   MediaErrorState mediaErrorState;
   if (mediaErrorState.hadException()) {
     mediaErrorState.raiseException(exceptionState);
     return;
   }
 
@@ skipping 662 matching lines @@
 DEFINE_TRACE(RTCPeerConnection) {
   visitor->trace(m_localStreams);
   visitor->trace(m_remoteStreams);
   visitor->trace(m_dispatchScheduledEventRunner);
   visitor->trace(m_scheduledEvents);
   EventTargetWithInlineData::trace(visitor);
   ActiveDOMObject::trace(visitor);
 }
 
 }  // namespace blink