commit 1358b5aefa0bc63c23a69f9a3119b72da6b3299d
author brandtr <brandtr@webrtc.org> Mon Dec 19 17:41:04 2016
committer Commit bot <commit-bot@chromium.org> Mon Dec 19 17:41:04 2016
tree 57caf50ece073b3c6dc3d85e7e93d8137a8c9310
parent 994a137871d99658d09ab8cafa47f9564ce929a1

Clean up storage of FlexFEC payload type in webrtc::VideoCodecSettings.

No need to pass a whole struct around, when only one member is used.

BUG=webrtc:5654

Review-Url: https://codereview.webrtc.org/2589833002
Cr-Original-Commit-Position: refs/heads/master@{#15687}
Cr-Mirrored-From: https://chromium.googlesource.com/external/webrtc
Cr-Mirrored-Commit: bb7066f9662a638f2f3b2f6100880ca595da0c3c

media/engine/webrtcvideoengine2.cc

diff --git a/media/engine/webrtcvideoengine2.cc b/media/engine/webrtcvideoengine2.cc
index 9e36697..9d9f141 100644
--- a/media/engine/webrtcvideoengine2.cc
+++ b/media/engine/webrtcvideoengine2.cc
@@ -1416,7 +1416,7 @@
     if (payload_type == codec.rtx_payload_type ||
         payload_type == codec.ulpfec.red_rtx_payload_type ||
         payload_type == codec.ulpfec.ulpfec_payload_type ||
-        payload_type == codec.flexfec.flexfec_payload_type) {
+        payload_type == codec.flexfec_payload_type) {
       return;
     }
   }
@@ -1771,7 +1771,7 @@
   }
   parameters_.config.rtp.ulpfec = codec_settings.ulpfec;
   parameters_.config.rtp.flexfec.flexfec_payload_type =
-      codec_settings.flexfec.flexfec_payload_type;
+      codec_settings.flexfec_payload_type;
 
   // Set RTX payload type if RTX is enabled.
   if (!parameters_.config.rtp.rtx.ssrcs.empty()) {
@@ -2258,7 +2258,8 @@
   // TODO(pbos): Reconfigure RTX based on incoming recv_codecs.
   config_.rtp.ulpfec = recv_codecs.front().ulpfec;
   flexfec_config_.flexfec_payload_type =
-      recv_codecs.front().flexfec.flexfec_payload_type;
+      recv_codecs.front().flexfec_payload_type;
+
   config_.rtp.nack.rtp_history_ms =
       HasNack(recv_codecs.begin()->codec) ? kNackHistoryMs : 0;
 }
@@ -2462,12 +2463,13 @@
 }
 
 WebRtcVideoChannel2::VideoCodecSettings::VideoCodecSettings()
-    : rtx_payload_type(-1) {}
+    : flexfec_payload_type(-1), rtx_payload_type(-1) {}
 
 bool WebRtcVideoChannel2::VideoCodecSettings::operator==(
     const WebRtcVideoChannel2::VideoCodecSettings& other) const {
   return codec == other.codec && ulpfec == other.ulpfec &&
-         flexfec == other.flexfec && rtx_payload_type == other.rtx_payload_type;
+         flexfec_payload_type == other.flexfec_payload_type &&
+         rtx_payload_type == other.rtx_payload_type;
 }
 
 bool WebRtcVideoChannel2::VideoCodecSettings::operator!=(
@@ -2568,7 +2570,7 @@
 
   for (size_t i = 0; i < video_codecs.size(); ++i) {
     video_codecs[i].ulpfec = ulpfec_config;
-    video_codecs[i].flexfec.flexfec_payload_type = flexfec_payload_type;
+    video_codecs[i].flexfec_payload_type = flexfec_payload_type;
     if (rtx_mapping[video_codecs[i].codec.id] != 0 &&
         rtx_mapping[video_codecs[i].codec.id] !=
             ulpfec_config.red_payload_type) {

media/engine/webrtcvideoengine2.h

diff --git a/media/engine/webrtcvideoengine2.h b/media/engine/webrtcvideoengine2.h
index 5e4233f..c7aeaae 100644
--- a/media/engine/webrtcvideoengine2.h
+++ b/media/engine/webrtcvideoengine2.h
@@ -192,7 +192,7 @@
 
     VideoCodec codec;
     webrtc::UlpfecConfig ulpfec;
-    webrtc::FlexfecConfig flexfec;
+    int flexfec_payload_type;
     int rtx_payload_type;
   };