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

Respects min ALR probing interval.

Browse Source 
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 <sprang@webrtc.org>
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26861}
This commit is contained in:
Sebastian Jansson
2019-02-26 15:29:53 +01:00
committed by Commit Bot
parent 1a16da1cf2
commit c9d0b08982
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
modules/congestion_controller/goog_cc/probe_controller.cc
View File

@ -285,8 +285,8 @@ std::vector<ProbeClusterConfig> 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);
}
}
}