Tweaked version of BBR for WebRTC.

webrtc/modules/remote_bitrate_estimator/test/estimators/congestion_window.cc

Index: webrtc/modules/remote_bitrate_estimator/test/estimators/congestion_window.cc
diff --git a/webrtc/modules/remote_bitrate_estimator/test/estimators/congestion_window.cc b/webrtc/modules/remote_bitrate_estimator/test/estimators/congestion_window.cc
index 3d357704b582f892a7429125a87593fb2927921f..3df12193b32be23fa597b636de1042116a5bac99 100644
--- a/webrtc/modules/remote_bitrate_estimator/test/estimators/congestion_window.cc
+++ b/webrtc/modules/remote_bitrate_estimator/test/estimators/congestion_window.cc
@@ -37,8 +37,6 @@ int CongestionWindow::GetCongestionWindow(BbrBweSender::Mode mode,
                                           int64_t bandwidth_estimate_bps,
                                           rtc::Optional<int64_t> min_rtt_ms,
                                           float gain) {
-  if (mode == BbrBweSender::PROBE_RTT)
-    return CongestionWindow::kMinimumCongestionWindowBytes;

[philipel, 2017/08/17 12:52:27]

Why don't we have a min congestion window any more?

[gnish1, 2017/08/17 15:37:49]

This implementation of BBR doesn't use minimum congestion window, but congestion
window gain of 0.65 instead.

   return GetTargetCongestionWindow(bandwidth_estimate_bps, min_rtt_ms, gain);
 }
 
@@ -57,14 +55,13 @@ int CongestionWindow::GetTargetCongestionWindow(
   // If we have no rtt sample yet, return the starting congestion window size.
   if (!min_rtt_ms)
     return gain * kStartingCongestionWindowBytes;
-  int bdp = *min_rtt_ms * bandwidth_estimate_bps;
+  int bdp = *min_rtt_ms * bandwidth_estimate_bps / 8000;

[philipel, 2017/08/17 12:52:27]

Why do we suddenly measure bandwidth delay product in kB?

[gnish1, 2017/08/17 15:37:49]

Because packet's are measured in bytes, so data_inflight is measured in bytes,
which means congestion window should be measured in bytes as well.

   int congestion_window = bdp * gain;
   // Congestion window could be zero in rare cases, when either no bandwidth
   // estimate is available, or path's min_rtt value is zero.
   if (!congestion_window)
     congestion_window = gain * kStartingCongestionWindowBytes;
-  return std::max(congestion_window,
-                  CongestionWindow::kMinimumCongestionWindowBytes);
+  return congestion_window;
 }
 }  // namespace bwe
 }  // namespace testing