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

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

Issue 2867713003: Remove hardcoded kValueSizeBytes values from variable-length header extensions. (Closed)
Patch Set: Created 3 years, 7 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/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 4aec05662fdaae852192570d7c9ca64dcb4a2776..36f0a06592c92c1112e8aa685732b2452003ece2 100644
--- a/webrtc/modules/rtp_rtcp/source/flexfec_sender.cc
+++ b/webrtc/modules/rtp_rtcp/source/flexfec_sender.cc
@@ -147,8 +147,9 @@ std::vector<std::unique_ptr<RtpPacketToSend>> FlexfecSender::GetFecPackets() {
}
// The overhead is BWE RTP header extensions and FlexFEC header.
-size_t FlexfecSender::MaxPacketOverhead() const {
- return rtp_header_extension_map_.GetTotalLengthInBytes() +
+size_t FlexfecSender::MaxPacketOverhead(
+ rtc::ArrayView<const RTPExtensionSizeInfo> extension_sizes) const {
+ return rtp_header_extension_map_.GetTotalLengthInBytes(extension_sizes) +
kFlexfecMaxHeaderSize;
}

Powered by Google App Engine
This is Rietveld 408576698