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

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

Issue 2542413002: Generalize FlexfecReceiveStream::Config. (CL1) (Closed)
Patch Set: Rebase. Created 4 years 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
« no previous file with comments | « webrtc/call/flexfec_receive_stream_unittest.cc ('k') | webrtc/media/engine/webrtcvideoengine2_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/media/engine/webrtcvideoengine2.cc
diff --git a/webrtc/media/engine/webrtcvideoengine2.cc b/webrtc/media/engine/webrtcvideoengine2.cc
index 92ac69801572e1e783fabfa89b917cbe4a790b6c..9e36697a6155b2c68ad355d89b892639b12488a5 100644
--- a/webrtc/media/engine/webrtcvideoengine2.cc
+++ b/webrtc/media/engine/webrtcvideoengine2.cc
@@ -2340,7 +2340,16 @@ void WebRtcVideoChannel2::WebRtcVideoReceiveStream::RecreateWebRtcStream() {
stream_ = call_->CreateVideoReceiveStream(config_.Copy());
stream_->Start();
if (IsFlexfecFieldTrialEnabled() && flexfec_config_.IsCompleteAndEnabled()) {
- flexfec_stream_ = call_->CreateFlexfecReceiveStream(flexfec_config_);
+ webrtc::FlexfecReceiveStream::Config config;
+ // Payload types and SSRCs come from the FlexFEC specific part of the SDP.
+ config.payload_type = flexfec_config_.flexfec_payload_type;
+ config.remote_ssrc = flexfec_config_.flexfec_ssrc;
+ config.protected_media_ssrcs = flexfec_config_.protected_media_ssrcs;
+ // RTCP messages and RTP header extensions apply to the entire track
+ // in the SDP.
+ config.transport_cc = config_.rtp.transport_cc;
+ config.extensions = config_.rtp.extensions;
+ flexfec_stream_ = call_->CreateFlexfecReceiveStream(config);
flexfec_stream_->Start();
}
}
« no previous file with comments | « webrtc/call/flexfec_receive_stream_unittest.cc ('k') | webrtc/media/engine/webrtcvideoengine2_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698