commit 9e17f56429ec340e347fc232708d6484f59fadac
author philipel <philipel@webrtc.org> Tue Sep 12 12:13:53 2017
committer Commit Bot <commit-bot@chromium.org> Tue Sep 12 12:13:53 2017
tree 90211e3796f03139840d39e3536eaacf8dd7f95d
parent 3661e7bb0e5cabf93e0b93058f5be84c01baf21d

Make sure send and receive deltas are positive for remote estimated probe clusters.

BUG=b/65531353

Review-Url: https://codereview.webrtc.org/3005393002
Cr-Original-Commit-Position: refs/heads/master@{#19796}
Cr-Mirrored-From: https://chromium.googlesource.com/external/webrtc
Cr-Mirrored-Commit: 0c011d9499deb10b3a3d15dd7a3c52b07babeacd

modules/remote_bitrate_estimator/remote_bitrate_estimator_abs_send_time.cc

diff --git a/modules/remote_bitrate_estimator/remote_bitrate_estimator_abs_send_time.cc b/modules/remote_bitrate_estimator/remote_bitrate_estimator_abs_send_time.cc
index d3dcfca..c43bc12 100644
--- a/modules/remote_bitrate_estimator/remote_bitrate_estimator_abs_send_time.cc
+++ b/modules/remote_bitrate_estimator/remote_bitrate_estimator_abs_send_time.cc
@@ -111,8 +111,11 @@
         ++current.num_above_min_delta;
       }
       if (!IsWithinClusterBounds(send_delta_ms, current)) {
-        if (current.count >= kMinClusterSize)
+        if (current.count >= kMinClusterSize &&
+            current.send_mean_ms > 0.0f &&
+            current.recv_mean_ms > 0.0f) {
           AddCluster(clusters, &current);
+        }
         current = Cluster();
       }
       current.send_mean_ms += send_delta_ms;
@@ -123,8 +126,11 @@
     prev_send_time = it->send_time_ms;
     prev_recv_time = it->recv_time_ms;
   }
-  if (current.count >= kMinClusterSize)
+  if (current.count >= kMinClusterSize &&
+      current.send_mean_ms > 0.0f &&
+      current.recv_mean_ms > 0.0f) {
     AddCluster(clusters, &current);
+  }
 }
 
 std::list<Cluster>::const_iterator