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

fix coding and documentary ambiguity in AimdRateControl::TimeToReduceFurther.

Browse Source 
BUG=webrtc:6812

Review-Url: https://codereview.webrtc.org/2549453002
Cr-Commit-Position: refs/heads/master@{#15383}
This commit is contained in:
howtofly
2016-12-02 03:27:08 -08:00
committed by Commit bot
parent a28a1b9db6
commit df28e47a4b
2 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
webrtc/modules/remote_bitrate_estimator/aimd_rate_control.cc
View File

@ -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,10 @@ 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;
// TODO(terelius/holmer): Investigate consequences of increasing
// the threshold to 0.95 * LatestEstimate().
const uint32_t threshold = static_cast<uint32_t> (0.5 * LatestEstimate());
return incoming_bitrate_bps < threshold;
}
return false;
}

2
webrtc/modules/remote_bitrate_estimator/aimd_rate_control.h
View File

@ -32,7 +32,7 @@ class AimdRateControl {
void SetMinBitrate(int min_bitrate_bps);
int64_t GetFeedbackInterval() const;
// Returns true if the bitrate estimate hasn't been changed for more than
// an RTT, or if the incoming_bitrate is more than 5% above the current
// an RTT, or if the incoming_bitrate is less than half of the current
// estimate. Should be used to decide if we should reduce the rate further
// when over-using.
bool TimeToReduceFurther(int64_t time_now,