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

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

Issue 2542413002: Generalize FlexfecReceiveStream::Config. (CL1) (Closed)
Patch Set: 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
Index: webrtc/media/engine/webrtcvideoengine2.cc
diff --git a/webrtc/media/engine/webrtcvideoengine2.cc b/webrtc/media/engine/webrtcvideoengine2.cc
index 9396fccb7deac6c2fedbe69b4fe2d66d0dcb84f7..1ce07b98a105eb411e4def506464995d115ccc50 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();
}
}

Powered by Google App Engine
This is Rietveld 408576698