From 8e75a523c8cecc71ba576c76e29b3ed8c039f8ed Mon Sep 17 00:00:00 2001 From: brandtr Date: Mon, 14 Nov 2016 04:07:24 -0800 Subject: [PATCH] Explicitly use RTX for RED in VideoQualityTest and video_loopback. After the removal of the RED/RTX workaround, we now need to explicitly enable RTX for RED. Prior to the removal of the workaround, RED over RTX was implicitly enabled whenever media over RTX was enabled. BUG=webrtc:6650 Review-Url: https://codereview.webrtc.org/2493723002 Cr-Commit-Position: refs/heads/master@{#15061} --- webrtc/video/video_quality_test.cc | 3 +++ 1 file changed, 3 insertions(+) diff --git a/webrtc/video/video_quality_test.cc b/webrtc/video/video_quality_test.cc index c80647da4b..a3a1cf8d74 100644 --- a/webrtc/video/video_quality_test.cc +++ b/webrtc/video/video_quality_test.cc @@ -1353,10 +1353,13 @@ void VideoQualityTest::RunWithRenderers(const Params& params) { if (params.video.fec) { video_send_config_.rtp.ulpfec.red_payload_type = kRedPayloadType; video_send_config_.rtp.ulpfec.ulpfec_payload_type = kUlpfecPayloadType; + video_send_config_.rtp.ulpfec.red_rtx_payload_type = kRtxRedPayloadType; video_receive_configs_[stream_id].rtp.ulpfec.red_payload_type = kRedPayloadType; video_receive_configs_[stream_id].rtp.ulpfec.ulpfec_payload_type = kUlpfecPayloadType; + video_receive_configs_[stream_id].rtp.ulpfec.red_rtx_payload_type = + kRtxRedPayloadType; } if (params_.screenshare.enabled)