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

BUG=b/65531353

Review-Url: https://codereview.webrtc.org/3005393002
Cr-Commit-Position: refs/heads/master@{#19796}
This commit is contained in:
philipel 2017-09-12 05:13:53 -07:00 committed by Commit Bot
parent bcc2176e64
commit 0c011d9499

View File

@ -111,8 +111,11 @@ void RemoteBitrateEstimatorAbsSendTime::ComputeClusters(
++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 @@ void RemoteBitrateEstimatorAbsSendTime::ComputeClusters(
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