From 1f99551775cd876c116d1d90cba94c8a4670d184 Mon Sep 17 00:00:00 2001 From: Jakob Ivarsson Date: Mon, 9 Nov 2020 17:55:56 +0100 Subject: [PATCH] Remove check for WebRTC-SendSideBwe-WithOverhead in bitrate controller. 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 Commit-Queue: Minyue Li Cr-Commit-Position: refs/heads/master@{#32570} --- .../audio_coding/audio_network_adaptor/bitrate_controller.cc | 5 ----- 1 file changed, 5 deletions(-) diff --git a/modules/audio_coding/audio_network_adaptor/bitrate_controller.cc b/modules/audio_coding/audio_network_adaptor/bitrate_controller.cc index eee6f403aa..40c8659069 100644 --- a/modules/audio_coding/audio_network_adaptor/bitrate_controller.cc +++ b/modules/audio_coding/audio_network_adaptor/bitrate_controller.cc @@ -53,11 +53,6 @@ void BitrateController::MakeDecision(AudioEncoderRuntimeConfig* config) { // Decision on |bitrate_bps| should not have been made. RTC_DCHECK(!config->bitrate_bps); 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) frame_length_ms_ = *config->frame_length_ms; int offset = config->last_fl_change_increase