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

Unified Diff: webrtc/modules/rtp_rtcp/source/flexfec_sender.cc

Issue 2490523002: Wire up FlexfecSender in RTPSenderVideo. (Closed)
Patch Set: Feedback response 2. Created 4 years, 1 month 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/modules/rtp_rtcp/source/flexfec_sender.cc
diff --git a/webrtc/modules/rtp_rtcp/source/flexfec_sender.cc b/webrtc/modules/rtp_rtcp/source/flexfec_sender.cc
index 55e8e5266919bbb978b44e8d5fcd1fff356549c1..4b0df09c4226bfc0f1dcea1a0cae150e0d347393 100644
--- a/webrtc/modules/rtp_rtcp/source/flexfec_sender.cc
+++ b/webrtc/modules/rtp_rtcp/source/flexfec_sender.cc
@@ -83,6 +83,13 @@ FlexfecSender::FlexfecSender(
<< ", is unsupported by FlexfecSender.";
}
}
+
+ // Calculate RTP header length.
+ RtpPacketToSend rtp_header(&rtp_header_extension_map_);
+ rtp_header.ReserveExtension<AbsoluteSendTime>();
+ rtp_header.ReserveExtension<TransmissionOffset>();
+ rtp_header.ReserveExtension<TransportSequenceNumber>();
+ rtp_header_length_ = rtp_header.headers_size();
danilchap 2016/11/09 15:58:46 that should be simpler: rtp_extra_header_length_ =
brandtr 2016/11/10 09:32:01 Nice, done.
}
FlexfecSender::~FlexfecSender() = default;

Powered by Google App Engine
This is Rietveld 408576698