Add config to specify raw audio priority bitrate including overhead.
Bug: webrtc:10815 Change-Id: I6a498d6c6bcd4fe4ba6ccc4d6f407d686528d946 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/145333 Commit-Queue: Christoffer Rodbro <crodbro@webrtc.org> Reviewed-by: Sebastian Jansson <srte@webrtc.org> Cr-Commit-Position: refs/heads/master@{#28590}
This commit is contained in:
committed by
Commit Bot
parent
3a51b0e9d1
commit
7f8dbe18ea
@ -9,6 +9,7 @@
|
||||
*/
|
||||
#include "rtc_base/experiments/audio_allocation_settings.h"
|
||||
|
||||
#include "rtc_base/logging.h"
|
||||
#include "system_wrappers/include/field_trial.h"
|
||||
|
||||
namespace webrtc {
|
||||
@ -29,10 +30,11 @@ AudioAllocationSettings::AudioAllocationSettings()
|
||||
min_bitrate_("min"),
|
||||
max_bitrate_("max"),
|
||||
priority_bitrate_("prio_rate", DataRate::Zero()),
|
||||
priority_bitrate_raw_("prio_rate_raw"),
|
||||
bitrate_priority_("rate_prio") {
|
||||
ParseFieldTrial(
|
||||
{&min_bitrate_, &max_bitrate_, &priority_bitrate_, &bitrate_priority_},
|
||||
field_trial::FindFullName("WebRTC-Audio-Allocation"));
|
||||
ParseFieldTrial({&min_bitrate_, &max_bitrate_, &priority_bitrate_,
|
||||
&priority_bitrate_raw_, &bitrate_priority_},
|
||||
field_trial::FindFullName("WebRTC-Audio-Allocation"));
|
||||
|
||||
// TODO(mflodman): Keep testing this and set proper values.
|
||||
// Note: This is an early experiment currently only supported by Opus.
|
||||
@ -40,6 +42,11 @@ AudioAllocationSettings::AudioAllocationSettings()
|
||||
constexpr int kMaxPacketSizeMs = WEBRTC_OPUS_SUPPORT_120MS_PTIME ? 120 : 60;
|
||||
min_overhead_bps_ = kOverheadPerPacket * 8 * 1000 / kMaxPacketSizeMs;
|
||||
}
|
||||
// priority_bitrate_raw will override priority_bitrate.
|
||||
if (priority_bitrate_raw_ && !priority_bitrate_->IsZero()) {
|
||||
RTC_LOG(LS_WARNING) << "'priority_bitrate' and '_raw' are mutually "
|
||||
"exclusive but both were configured.";
|
||||
}
|
||||
}
|
||||
|
||||
AudioAllocationSettings::~AudioAllocationSettings() {}
|
||||
@ -111,6 +118,9 @@ absl::optional<DataRate> AudioAllocationSettings::MaxBitrate() const {
|
||||
}
|
||||
DataRate AudioAllocationSettings::DefaultPriorityBitrate() const {
|
||||
DataRate max_overhead = DataRate::Zero();
|
||||
if (priority_bitrate_raw_) {
|
||||
return *priority_bitrate_raw_;
|
||||
}
|
||||
if (send_side_bwe_with_overhead_) {
|
||||
const TimeDelta kMinPacketDuration = TimeDelta::ms(20);
|
||||
max_overhead = DataSize::bytes(kOverheadPerPacket) / kMinPacketDuration;
|
||||
|
||||
Reference in New Issue
Block a user