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 34b8242ed760dcc96e52bb7d870a926c0d472885..4cdd346ed83ed8a592e66989814553ec69483853 100644 |
--- a/webrtc/modules/congestion_controller/delay_based_bwe.cc |
+++ b/webrtc/modules/congestion_controller/delay_based_bwe.cc |
@@ -35,6 +35,7 @@ constexpr double kTimestampToMs = |
// This ssrc is used to fulfill the current API but will be removed |
// after the API has been changed. |
constexpr uint32_t kFixedSsrc = 0; |
+constexpr int64_t kDefaultRttMs = 200; |
} // namespace |
namespace webrtc { |
@@ -47,7 +48,8 @@ DelayBasedBwe::DelayBasedBwe(Clock* clock) |
incoming_bitrate_(kBitrateWindowMs, 8000), |
last_update_ms_(-1), |
last_seen_packet_ms_(-1), |
- uma_recorded_(false) { |
+ uma_recorded_(false), |
+ rtt_ms_(kDefaultRttMs) { |
network_thread_.DetachFromThread(); |
} |
@@ -132,8 +134,7 @@ DelayBasedBwe::Result DelayBasedBwe::IncomingPacketInfo( |
rtc::Optional<uint32_t> incoming_rate = |
incoming_bitrate_.Rate(info.arrival_time_ms); |
if (!result.updated && |
- (last_update_ms_ == -1 || |
- now_ms - last_update_ms_ > remote_rate_.GetFeedbackInterval())) { |
+ (last_update_ms_ == -1 || now_ms - last_update_ms_ > rtt_ms_)) { |
stefan-webrtc
2016/10/17 18:59:49
Do you think there's a true value having this chec
|
result.updated = UpdateEstimate(info.arrival_time_ms, now_ms, |
&result.target_bitrate_bps); |
} |
@@ -158,6 +159,7 @@ bool DelayBasedBwe::UpdateEstimate(int64_t arrival_time_ms, |
} |
void DelayBasedBwe::OnRttUpdate(int64_t avg_rtt_ms, int64_t max_rtt_ms) { |
+ rtt_ms_ = avg_rtt_ms; |
remote_rate_.SetRtt(avg_rtt_ms); |
} |