Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(134)

Unified Diff: webrtc/media/engine/webrtcvideoengine2.cc

Issue 2654163006: Reland of Make RTX pt/apt reconfigurable by calling WebRtcVideoChannel2::SetRecvParameters. (Closed)
Patch Set: Update transition code. Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/media/engine/webrtcvideoengine2.cc
diff --git a/webrtc/media/engine/webrtcvideoengine2.cc b/webrtc/media/engine/webrtcvideoengine2.cc
index 0cd31937cc3e07215b7d6621d58c4cf4c0f11562..40f486dc0f9aa62291e046d952c89e9adad760ef 100644
--- a/webrtc/media/engine/webrtcvideoengine2.cc
+++ b/webrtc/media/engine/webrtcvideoengine2.cc
@@ -1233,16 +1233,7 @@ void WebRtcVideoChannel2::ConfigureReceiverRtp(
flexfec_config->rtp_header_extensions = config->rtp.extensions;
}
- for (size_t i = 0; i < recv_codecs_.size(); ++i) {
- uint32_t rtx_ssrc;
- if (recv_codecs_[i].rtx_payload_type != -1 &&
- sp.GetFidSsrc(ssrc, &rtx_ssrc)) {
- webrtc::VideoReceiveStream::Config::Rtp::Rtx& rtx =
- config->rtp.rtx[recv_codecs_[i].codec.id];
- rtx.ssrc = rtx_ssrc;
- rtx.payload_type = recv_codecs_[i].rtx_payload_type;
- }
- }
+ sp.GetFidSsrc(ssrc, &config->rtp.rtx_ssrc);
config->rtp.extensions = recv_rtp_extensions_;
}
@@ -2205,7 +2196,12 @@ void WebRtcVideoChannel2::WebRtcVideoReceiveStream::ConfigureCodecs(
config_.decoders.push_back(decoder);
}
- // TODO(pbos): Reconfigure RTX based on incoming recv_codecs.
+ config_.rtp.rtx_payload_types.clear();
+ for (const VideoCodecSettings& recv_codec : recv_codecs) {
+ config_.rtp.rtx_payload_types[recv_codec.codec.id] =
+ recv_codec.rtx_payload_type;
+ }
+
config_.rtp.ulpfec = recv_codecs.front().ulpfec;
flexfec_config_.payload_type = recv_codecs.front().flexfec_payload_type;
« no previous file with comments | « webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc ('k') | webrtc/media/engine/webrtcvideoengine2_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698