From c9d0b0898290d2efec33b6b355afb1a452ed031f Mon Sep 17 00:00:00 2001 From: Sebastian Jansson Date: Tue, 26 Feb 2019 15:29:53 +0100 Subject: [PATCH] Respects min ALR probing interval. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit In a previous refactor, the ALR probe timestamp update was moved after a return statement by accident. This CL fixes this. The impact of this bug is limited as there are several other criteria that has to be fulfilled for sending ALR probes. Bug: None Change-Id: Ia85e6ff9d782c1c4722a3df7e01ed803cf86b11d Reviewed-on: https://webrtc-review.googlesource.com/c/124489 Reviewed-by: Erik Språng Commit-Queue: Sebastian Jansson Cr-Commit-Position: refs/heads/master@{#26861} --- modules/congestion_controller/goog_cc/probe_controller.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modules/congestion_controller/goog_cc/probe_controller.cc b/modules/congestion_controller/goog_cc/probe_controller.cc index 392fddd995..d28c189b7b 100644 --- a/modules/congestion_controller/goog_cc/probe_controller.cc +++ b/modules/congestion_controller/goog_cc/probe_controller.cc @@ -285,8 +285,8 @@ std::vector ProbeController::RequestProbe( RTC_HISTOGRAM_COUNTS_10000( "WebRTC.BWE.BweDropProbingIntervalInS", (at_time_ms - last_bwe_drop_probing_time_ms_) / 1000); - return InitiateProbing(at_time_ms, {suggested_probe_bps}, false); last_bwe_drop_probing_time_ms_ = at_time_ms; + return InitiateProbing(at_time_ms, {suggested_probe_bps}, false); } } }