zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove check for WebRTC-SendSideBwe-WithOverhead in bitrate controller.

Browse Source 
This was default enabled in:
https://webrtc-review.googlesource.com/c/src/+/191221

Bug: webrtc:6762
Change-Id: Ic8054136ed854866c0fdc71f9d747b8abcd9200e
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/192381
Reviewed-by: Minyue Li <minyue@webrtc.org>
Commit-Queue: Minyue Li <minyue@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#32570}
This commit is contained in:
Jakob Ivarsson
2020-11-09 17:55:56 +01:00
committed by Commit Bot
parent a094787cb7
commit 1f99551775
1 changed files with 0 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
modules/audio_coding/audio_network_adaptor/bitrate_controller.cc
View File

@ -53,11 +53,6 @@ void BitrateController::MakeDecision(AudioEncoderRuntimeConfig* config) {
// Decision on |bitrate_bps| should not have been made. // Decision on |bitrate_bps| should not have been made.
RTC_DCHECK(!config->bitrate_bps); RTC_DCHECK(!config->bitrate_bps);
if (target_audio_bitrate_bps_ && overhead_bytes_per_packet_) { if (target_audio_bitrate_bps_ && overhead_bytes_per_packet_) {
// Current implementation of BitrateController can only work when
// |metrics.target_audio_bitrate_bps| includes overhead is enabled. This is
// currently governed by the following field trial.
RTC_DCHECK(
webrtc::field_trial::IsEnabled("WebRTC-SendSideBwe-WithOverhead"));
if (config->frame_length_ms) if (config->frame_length_ms)
frame_length_ms_ = *config->frame_length_ms; frame_length_ms_ = *config->frame_length_ms;
int offset = config->last_fl_change_increase int offset = config->last_fl_change_increase