Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(200)

Unified Diff: webrtc/modules/congestion_controller/delay_based_bwe.cc

Issue 2789233005: Move BWE period calculation from ProbingIntervalEstimator to AimdRateControl. (Closed)
Patch Set: Nits Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/congestion_controller/delay_based_bwe.cc
diff --git a/webrtc/modules/congestion_controller/delay_based_bwe.cc b/webrtc/modules/congestion_controller/delay_based_bwe.cc
index 7b19685d99f56b68d6922030abb0554956896b8d..c091cb9a81721acc7ba78eb892bc5a4578eadb7a 100644
--- a/webrtc/modules/congestion_controller/delay_based_bwe.cc
+++ b/webrtc/modules/congestion_controller/delay_based_bwe.cc
@@ -160,7 +160,6 @@ DelayBasedBwe::DelayBasedBwe(RtcEventLog* event_log, const Clock* clock)
trendline_window_size_(kDefaultTrendlineWindowSize),
trendline_smoothing_coeff_(kDefaultTrendlineSmoothingCoeff),
trendline_threshold_gain_(kDefaultTrendlineThresholdGain),
- probing_interval_estimator_(&rate_control_),
consecutive_delayed_feedbacks_(0),
last_logged_bitrate_(0),
last_logged_state_(BandwidthUsage::kBwNormal) {
@@ -363,7 +362,7 @@ void DelayBasedBwe::SetMinBitrate(int min_bitrate_bps) {
rate_control_.SetMinBitrate(min_bitrate_bps);
}
-int64_t DelayBasedBwe::GetProbingIntervalMs() const {
- return probing_interval_estimator_.GetIntervalMs();
+int64_t DelayBasedBwe::GetExpectedBwePeriodMs() const {
+ return rate_control_.GetExpectedBandwidthPeriodMs();
}
} // namespace webrtc

Powered by Google App Engine
This is Rietveld 408576698