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

Remove WebRTC-Pacer-DynamicPaddingTarget field trial

Browse Source 
Bug: webrtc:10809
Change-Id: I9f4adbea5721408b339bbf7497c20834537e50c0
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/268145
Auto-Submit: Erik Språng <sprang@webrtc.org>
Reviewed-by: Philip Eliasson <philipel@webrtc.org>
Commit-Queue: Philip Eliasson <philipel@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#37480}
This commit is contained in:
Erik Språng
2022-07-07 12:41:57 +02:00
committed by WebRTC LUCI CQ
parent b858d3f53e
commit c52e627c83
3 changed files with 5 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
modules/pacing/pacing_controller.cc
View File

@ -35,6 +35,7 @@ constexpr TimeDelta kCongestedPacketInterval = TimeDelta::Millis(500);
// The maximum debt level, in terms of time, capped when sending packets.
constexpr TimeDelta kMaxDebtInTime = TimeDelta::Millis(500);
constexpr TimeDelta kMaxElapsedTime = TimeDelta::Seconds(2);
constexpr TimeDelta kTargetPaddingDuration = TimeDelta::Millis(5);
bool IsDisabled(const FieldTrialsView& field_trials, absl::string_view key) {
return absl::StartsWith(field_trials.Lookup(key), "Disabled");
@ -44,14 +45,6 @@ bool IsEnabled(const FieldTrialsView& field_trials, absl::string_view key) {
return absl::StartsWith(field_trials.Lookup(key), "Enabled");
}
TimeDelta GetDynamicPaddingTarget(const FieldTrialsView& field_trials) {
FieldTrialParameter<TimeDelta> padding_target("timedelta",
TimeDelta::Millis(5));
ParseFieldTrial({&padding_target},
field_trials.Lookup("WebRTC-Pacer-DynamicPaddingTarget"));
return padding_target.Get();
}
std::unique_ptr<PacingController::PacketQueue> CreatePacketQueue(
const FieldTrialsView& field_trials,
Timestamp creation_time) {
@ -85,7 +78,6 @@ PacingController::PacingController(Clock* clock,
pace_audio_(IsEnabled(field_trials_, "WebRTC-Pacer-BlockAudio")),
ignore_transport_overhead_(
IsEnabled(field_trials_, "WebRTC-Pacer-IgnoreTransportOverhead")),
padding_target_duration_(GetDynamicPaddingTarget(field_trials_)),
min_packet_limit_(kDefaultMinPacketLimit),
transport_overhead_per_packet_(DataSize::Zero()),
send_burst_interval_(TimeDelta::Zero()),
@ -571,7 +563,7 @@ DataSize PacingController::PaddingToAdd(DataSize recommended_probe_size,
}
if (padding_rate_ > DataRate::Zero() && padding_debt_ == DataSize::Zero()) {
return padding_target_duration_ * padding_rate_;
return kTargetPaddingDuration * padding_rate_;
}
return DataSize::Zero();
}

3
modules/pacing/pacing_controller.h
View File

@ -229,9 +229,6 @@ class PacingController {
const bool send_padding_if_silent_;
const bool pace_audio_;
const bool ignore_transport_overhead_;
// In dynamic mode, indicates the target size when requesting padding,
// expressed as a duration in order to adjust for varying padding rate.
const TimeDelta padding_target_duration_;
TimeDelta min_packet_limit_;
DataSize transport_overhead_per_packet_;

9
modules/pacing/pacing_controller_unittest.cc
View File

@ -1812,14 +1812,11 @@ TEST_F(PacingControllerTest, NextSendTimeAccountsForPadding) {
}
TEST_F(PacingControllerTest, PaddingTargetAccountsForPaddingRate) {
// Re-init pacer with an explicitly set padding target of 10ms;
const TimeDelta kPaddingTarget = TimeDelta::Millis(10);
ExplicitKeyValueConfig field_trials(
"WebRTC-Pacer-DynamicPaddingTarget/timedelta:10ms/");
// Target size for a padding packet is 5ms * padding rate.
const TimeDelta kPaddingTarget = TimeDelta::Millis(5);
srand(0);
// Need to initialize PacingController after we initialize clock.
auto pacer =
std::make_unique<PacingController>(&clock_, &callback_, field_trials);
auto pacer = std::make_unique<PacingController>(&clock_, &callback_, trials_);
const uint32_t kSsrc = 12345;
const DataRate kPacingDataRate = DataRate::KilobitsPerSec(125);