commit b939d35e8ec3b044ba1f3e58188d9eeb0d89c364
author Sebastian Jansson <srte@webrtc.org> Fri Nov 30 07:59:00 2018
committer Commit Bot <commit-bot@chromium.org> Fri Nov 30 09:44:55 2018
tree dd0dfe2f912845e6ed8ce64e033ec409cb7ddb85
parent 0c3f4d37090bdbc892e4f26716a4415233f4d747

Fixes DCHECK bug in LinkCapacityEstimator.

Conversion to kbps will fail if the estimate is lower than the deviation
estimate * 3, since that would produce a negative value.

Bug: webrtc:9718
Change-Id: I83b52acd476d90b1f22c9db9894fa26c9a3e8e17
Reviewed-on: https://webrtc-review.googlesource.com/c/112560
Reviewed-by: Niels Moller <nisse@webrtc.org>
Reviewed-by: Sebastian Jansson <srte@webrtc.org>
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25854}

modules/congestion_controller/goog_cc/link_capacity_estimator.cc

diff --git a/modules/congestion_controller/goog_cc/link_capacity_estimator.cc b/modules/congestion_controller/goog_cc/link_capacity_estimator.cc
index d16f074..8f59c9a 100644
--- a/modules/congestion_controller/goog_cc/link_capacity_estimator.cc
+++ b/modules/congestion_controller/goog_cc/link_capacity_estimator.cc
@@ -25,8 +25,8 @@
 
 DataRate LinkCapacityEstimator::LowerBound() const {
   if (estimate_kbps_.has_value())
-    return DataRate::kbps(estimate_kbps_.value() -
-                          3 * deviation_estimate_kbps());
+    return DataRate::kbps(
+        std::max(0.0, estimate_kbps_.value() - 3 * deviation_estimate_kbps()));
   return DataRate::Zero();
 }