aboutsummaryrefslogtreecommitdiff
path: root/modules/pacing
diff options
context:
space:
mode:
authorSebastian Jansson <srte@webrtc.org>2019-01-23 12:37:49 +0100
committerCommit Bot <commit-bot@chromium.org>2019-01-23 12:13:29 +0000
commit470a5eae9306f8b2b95613e5df9f64aa0438ebd1 (patch)
treebeec8569413befa79eaa633d0358957803488596 /modules/pacing
parent33b716f7ddb98c2c14be750b4243e49ec0fd31d5 (diff)
downloadwebrtc-470a5eae9306f8b2b95613e5df9f64aa0438ebd1.tar.gz
Introduces common AudioAllocationSettings class.
This class collects the field trial based configuration of audio allocation and bandwidth in one place. This makes it easier overview and prepares for future cleanup of the trials. Bug: webrtc:9718 Change-Id: I34a441c0165b423f1e2ee63894337484684146ac Reviewed-on: https://webrtc-review.googlesource.com/c/118282 Commit-Queue: Sebastian Jansson <srte@webrtc.org> Reviewed-by: Karl Wiberg <kwiberg@webrtc.org> Reviewed-by: Oskar Sundbom <ossu@webrtc.org> Cr-Commit-Position: refs/heads/master@{#26370}
Diffstat (limited to 'modules/pacing')
-rw-r--r--modules/pacing/paced_sender.cc5
-rw-r--r--modules/pacing/paced_sender.h2
2 files changed, 3 insertions, 4 deletions
diff --git a/modules/pacing/paced_sender.cc b/modules/pacing/paced_sender.cc
index 1a160d854c..9ab8cdb585 100644
--- a/modules/pacing/paced_sender.cc
+++ b/modules/pacing/paced_sender.cc
@@ -51,7 +51,7 @@ PacedSender::PacedSender(const Clock* clock,
drain_large_queues_(!field_trial::IsDisabled("WebRTC-Pacer-DrainQueue")),
send_padding_if_silent_(
field_trial::IsEnabled("WebRTC-Pacer-PadInSilence")),
- video_blocks_audio_(!field_trial::IsDisabled("WebRTC-Pacer-BlockAudio")),
+ pace_audio_(!field_trial::IsDisabled("WebRTC-Pacer-BlockAudio")),
min_packet_limit_ms_("", kDefaultMinPacketLimitMs),
last_timestamp_ms_(clock_->TimeInMilliseconds()),
paused_(false),
@@ -408,8 +408,7 @@ const RoundRobinPacketQueue::Packet* PacedSender::GetPendingPacket(
// reinsert it if send fails.
const RoundRobinPacketQueue::Packet* packet = &packets_.BeginPop();
bool audio_packet = packet->priority == kHighPriority;
- bool apply_pacing =
- !audio_packet || account_for_audio_ || video_blocks_audio_;
+ bool apply_pacing = !audio_packet || pace_audio_;
if (apply_pacing && (Congested() || (media_budget_.bytes_remaining() == 0 &&
pacing_info.probe_cluster_id ==
PacedPacketInfo::kNotAProbe))) {
diff --git a/modules/pacing/paced_sender.h b/modules/pacing/paced_sender.h
index 1949dc00ed..0c7a90cc60 100644
--- a/modules/pacing/paced_sender.h
+++ b/modules/pacing/paced_sender.h
@@ -174,7 +174,7 @@ class PacedSender : public Pacer {
const bool drain_large_queues_;
const bool send_padding_if_silent_;
- const bool video_blocks_audio_;
+ const bool pace_audio_;
FieldTrialParameter<int> min_packet_limit_ms_;
rtc::CriticalSection critsect_;