Don't do reconfiguration if recv codec order/preference changes

talk/media/webrtc/webrtcvideoengine2.h

Index: talk/media/webrtc/webrtcvideoengine2.h
diff --git a/talk/media/webrtc/webrtcvideoengine2.h b/talk/media/webrtc/webrtcvideoengine2.h
index 9d04526f4b33d305512c78c0960908876c1093eb..80f2043f9c5cc9b9f7f461c0f774e5ee2e192e1a 100644
--- a/talk/media/webrtc/webrtcvideoengine2.h
+++ b/talk/media/webrtc/webrtcvideoengine2.h
@@ -261,6 +261,9 @@ class WebRtcVideoChannel2 : public rtc::MessageHandler,
     int rtx_payload_type;
   };
 
+  static std::string CodecSettingsVectorToString(
+      const std::vector<VideoCodecSettings>& codecs);
+
   // Wrapper for the sender part, this is where the capturer is connected and
   // frames are then converted from cricket frames to webrtc frames.
   class WebRtcVideoSendStream : public sigslot::has_slots<> {
@@ -494,6 +497,8 @@ class WebRtcVideoChannel2 : public rtc::MessageHandler,
       const std::vector<VideoCodec>& codecs);
   std::vector<VideoCodecSettings> FilterSupportedCodecs(
       const std::vector<VideoCodecSettings>& mapped_codecs) const;
+  static bool CodecsHaveChanged(std::vector<VideoCodecSettings> before,
+                                std::vector<VideoCodecSettings> after);

[pthatcher1, 2015/08/18 20:24:32]

This should be ReceiveCodecsHaveChanged, since the order/preference does matter
for send codecs.

[Taylor Brandstetter, 2015/08/20 23:07:47]

Done.

 
   void FillSenderStats(VideoMediaInfo* info);
   void FillReceiverStats(VideoMediaInfo* info);