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

Unified Diff: webrtc/modules/remote_bitrate_estimator/aimd_rate_control.cc

Issue 2537043003: fix wrong implementation of AimdRateControl::TimeToReduceFurther. (Closed)
Patch Set: fix wrong implementation of AimdRateControl::TimeToReduceFurther Created 4 years 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/remote_bitrate_estimator/aimd_rate_control.cc
diff --git a/webrtc/modules/remote_bitrate_estimator/aimd_rate_control.cc b/webrtc/modules/remote_bitrate_estimator/aimd_rate_control.cc
index 530453d74cc1612c105de25bb7cc0280d261c53a..0e1ce16cf672a963b8518ebfedce0f77ea275ce2 100644
--- a/webrtc/modules/remote_bitrate_estimator/aimd_rate_control.cc
+++ b/webrtc/modules/remote_bitrate_estimator/aimd_rate_control.cc
@@ -24,7 +24,6 @@
namespace webrtc {
static const int64_t kDefaultRttMs = 200;
-static const double kWithinIncomingBitrateHysteresis = 1.05;
static const int64_t kMaxFeedbackIntervalMs = 1000;
AimdRateControl::AimdRateControl()
@@ -72,10 +71,9 @@ bool AimdRateControl::TimeToReduceFurther(int64_t time_now,
return true;
}
if (ValidEstimate()) {
- const int threshold = static_cast<int>(kWithinIncomingBitrateHysteresis *
- incoming_bitrate_bps);
- const int bitrate_difference = LatestEstimate() - incoming_bitrate_bps;
- return bitrate_difference > threshold;
+ const uint32_t threshold = static_cast<uint32_t>
+ (0.95 * LatestEstimate());
+ return incoming_bitrate_bps <= threshold;
}
return false;
}

Powered by Google App Engine
This is Rietveld 408576698