Only generate one CNAME per PeerConnection.

webrtc/pc/mediasession.cc

Index: webrtc/pc/mediasession.cc
diff --git a/webrtc/pc/mediasession.cc b/webrtc/pc/mediasession.cc
index a9d1b95c22aee6aae9299315a9ecc05586242336..1b1cb8e186dbb812b27b68b05dc96c62961936bb 100644
--- a/webrtc/pc/mediasession.cc
+++ b/webrtc/pc/mediasession.cc
@@ -232,11 +232,9 @@ static bool GenerateCname(const StreamParamsVec& params_vec,
   if (!cname)
     return false;
 
-  // Check if a CNAME exist for any of the other synched streams.
+  // Check if a CNAME exist for any of the other streams.

[pthatcher1, 2016/04/12 22:43:14]

exist => exists

   for (MediaSessionOptions::Streams::const_iterator stream_it = streams.begin();
        stream_it != streams.end() ; ++stream_it) {

[pthatcher1, 2016/04/12 22:43:14]

While you're in here, can you update this to a c++11-style for loop?

for (MediaSessionOptions::Stream stream : streams)

-    if (synch_label != stream_it->sync_label)
-      continue;
 
     // groupid is empty for StreamParams generated using
     // MediaSessionDescriptionFactory.