Add DCHECK for wrap around in RtpVideoSender::OnBitrateUpdated.

Bug: webrtc:7510
Change-Id: Idfe645aa75cf6a0699caa94063f47c57c2ed5ee2
Reviewed-on: https://webrtc-review.googlesource.com/c/107728
Commit-Queue: Björn Terelius <terelius@webrtc.org>
Reviewed-by: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25356}
This commit is contained in:
Bjorn Terelius 2018-10-25 11:07:29 +02:00 committed by Commit Bot
parent 370bae466c
commit 2506839dba

View File

@ -600,12 +600,14 @@ void RtpVideoSender::OnBitrateUpdated(uint32_t bitrate_bps,
rtc::CritScope lock(&crit_);
uint32_t payload_bitrate_bps = bitrate_bps;
if (send_side_bwe_with_overhead_) {
payload_bitrate_bps -= CalculateOverheadRateBps(
uint32_t overhead_bps = CalculateOverheadRateBps(
CalculatePacketRate(
bitrate_bps,
rtp_config_.max_packet_size + transport_overhead_bytes_per_packet_),
overhead_bytes_per_packet_ + transport_overhead_bytes_per_packet_,
bitrate_bps);
RTC_DCHECK_LE(overhead_bps, bitrate_bps);
payload_bitrate_bps = bitrate_bps - overhead_bps;
}
// Get the encoder target rate. It is the estimated network rate -