From 01ab084f4770ee3f075136f3f04f53d4a4fd7ec3 Mon Sep 17 00:00:00 2001 From: Jakob Ivarsson Date: Fri, 6 Mar 2020 09:59:56 +0100 Subject: [PATCH] Add minimum overhead to configured priorty bitrate instead of maximum. This makes an assumption that if we have variable frame length then we will increase payload bitrate up to priority bitrate before adapting the frame length. Bug: webrtc:11001 Change-Id: Iec51d5ccce053d55ccd30a9e4712765227e10852 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/169852 Reviewed-by: Sebastian Jansson Reviewed-by: Sam Zackrisson Commit-Queue: Jakob Ivarsson Cr-Commit-Position: refs/heads/master@{#30713} --- audio/audio_send_stream.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/audio/audio_send_stream.cc b/audio/audio_send_stream.cc index 5bb2e858c6..c5e2c50026 100644 --- a/audio/audio_send_stream.cc +++ b/audio/audio_send_stream.cc @@ -828,8 +828,8 @@ void AudioSendStream::ConfigureBitrateObserver() { RTC_DCHECK(frame_length_range_); const DataSize kOverheadPerPacket = DataSize::Bytes(total_packet_overhead_bytes_); - DataRate max_overhead = kOverheadPerPacket / frame_length_range_->first; - priority_bitrate += max_overhead; + DataRate min_overhead = kOverheadPerPacket / frame_length_range_->second; + priority_bitrate += min_overhead; } } if (allocation_settings_.priority_bitrate_raw)