From 3f6bf3a4ab161c0639e550a50712810d08240a44 Mon Sep 17 00:00:00 2001 From: Evan Shrubsole Date: Wed, 20 Feb 2019 10:00:40 +0100 Subject: [PATCH] Clarify that pacing rate is based on raw target rate Change-Id: Ib9cc068c88f45536ea5d9d0a84fab9da8f963131 Bug: None Reviewed-on: https://webrtc-review.googlesource.com/c/123050 Commit-Queue: Sebastian Jansson Reviewed-by: Sebastian Jansson Reviewed-by: Christoffer Rodbro Cr-Commit-Position: refs/heads/master@{#26765} --- .../congestion_controller/goog_cc/goog_cc_network_control.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/modules/congestion_controller/goog_cc/goog_cc_network_control.cc b/modules/congestion_controller/goog_cc/goog_cc_network_control.cc index af8594d21e..c5466f7a43 100644 --- a/modules/congestion_controller/goog_cc/goog_cc_network_control.cc +++ b/modules/congestion_controller/goog_cc/goog_cc_network_control.cc @@ -633,6 +633,8 @@ void GoogCcNetworkController::MaybeTriggerOnNetworkChanged( } PacerConfig GoogCcNetworkController::GetPacingRates(Timestamp at_time) const { + // Pacing rate is based on target rate before congestion window pushback, + // because we don't want to build queues in the pacer when pushback occurs. DataRate pacing_rate = std::max(min_pacing_rate_, last_raw_target_rate_) * pacing_factor_; DataRate padding_rate =