Implement periodic bandwidth probing in application-limited region.

webrtc/modules/pacing/alr_detector.cc

Index: webrtc/modules/pacing/alr_detector.cc
diff --git a/webrtc/modules/pacing/alr_detector.cc b/webrtc/modules/pacing/alr_detector.cc
index 90afe18fef76a3368d1db725e758c5d31ffee8e9..1d771ac4fedf6fea492ce936c0fe508ebb672c56 100644
--- a/webrtc/modules/pacing/alr_detector.cc
+++ b/webrtc/modules/pacing/alr_detector.cc
@@ -44,10 +44,10 @@ void AlrDetector::OnBytesSent(size_t bytes_sent, int64_t now_ms) {
     return;
 
   int percentage = static_cast<int>(*rate) * 100 / estimated_bitrate_bps_;
-  if (percentage < kAlrStartUsagePercent && !application_limited_) {
-    application_limited_ = true;
-  } else if (percentage > kAlrEndUsagePercent && application_limited_) {
-    application_limited_ = false;
+  if (percentage < kAlrStartUsagePercent && !alr_started_time_ms_) {
+    alr_started_time_ms_ = now_ms;
+  } else if (percentage > kAlrEndUsagePercent && alr_started_time_ms_) {
+    alr_started_time_ms_ = 0;
   }
 }
 
@@ -56,8 +56,9 @@ void AlrDetector::SetEstimatedBitrate(int bitrate_bps) {
   estimated_bitrate_bps_ = bitrate_bps;
 }
 
-bool AlrDetector::InApplicationLimitedRegion() const {
-  return application_limited_;
+rtc::Optional<int64_t> AlrDetector::GetApplicationLimitedRegionStartTime()
+    const {
+  return rtc::Optional<int64_t>(alr_started_time_ms_);

[philipel, 2016/11/22 10:55:10]

We always return a value? That doesn't look right.

if (alr_started_time_ms_ == 0)
  return rtc::Optional<int64_t>();
return rtc::Optional<int64_t>(alr_started_time_ms_);

[Sergey Ulanov, 2016/11/22 23:21:00]

Done.

 }
 
 }  // namespace webrtc