Only use payload size within the receive/send interval for bitrate probing.

webrtc/modules/congestion_controller/probe_bitrate_estimator.cc

 /*
  *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 30 matching lines @@
 ProbingResult ProbeBitrateEstimator::PacketFeedback(
     const PacketInfo& packet_info) {
   // If this is not a probing packet or if this probing packet
   // belongs to an old cluster, do nothing.
   if (packet_info.probe_cluster_id == PacketInfo::kNotAProbe ||
       packet_info.probe_cluster_id < last_valid_cluster_id_) {
     return ProbingResult();
   }
 
   AggregatedCluster* cluster = &clusters_[packet_info.probe_cluster_id];
+
   cluster->first_send_ms =
       std::min(cluster->first_send_ms, packet_info.send_time_ms);
   cluster->last_send_ms =
       std::max(cluster->last_send_ms, packet_info.send_time_ms);
   cluster->first_receive_ms =
       std::min(cluster->first_receive_ms, packet_info.arrival_time_ms);
   cluster->last_receive_ms =
       std::max(cluster->last_receive_ms, packet_info.arrival_time_ms);
   cluster->size += packet_info.payload_size * 8;
-  cluster->num_probes += 1;
+  cluster->send_times_ms.insert(packet_info.send_time_ms);

[danilchap, 2016/08/19 14:08:30]

why is it needed?

 
   // Clean up old clusters.
   while (clusters_.size() > kMaxNumSavedClusters)
     clusters_.erase(clusters_.begin());
 
-  if (cluster->num_probes < kMinNumProbesValidCluster)
+  if (cluster->send_times_ms.size() < kMinNumProbesValidCluster)

[danilchap, 2016/08/19 14:08:30]

If you worry two small padding packet is not enough to do the probing
calculation, may be better expand this if instead:
if (cluster->num_probes < kMinProbes || cluster->size < kMinSize)

     return ProbingResult();
 
   float send_interval_ms = cluster->last_send_ms - cluster->first_send_ms;
   float receive_interval_ms =
       cluster->last_receive_ms - cluster->first_receive_ms;
 
   // Since the send/receive interval does not include the send/receive time of
   // the last/first packet we expand the interval by the average inverval
   // between the probing packets.
   float interval_correction =
-      static_cast<float>(cluster->num_probes) / (cluster->num_probes - 1);
+      static_cast<float>(cluster->send_times_ms.size()) /
+      (cluster->send_times_ms.size() - 1);
   send_interval_ms *= interval_correction;
   receive_interval_ms *= interval_correction;
 
   if (send_interval_ms == 0 || receive_interval_ms == 0) {
     LOG(LS_INFO) << "Probing unsuccessful, invalid send/receive interval"
                  << " [cluster id: " << packet_info.probe_cluster_id
                  << "] [send interval: " << send_interval_ms << " ms]"
                  << " [receive interval: " << receive_interval_ms << " ms]";
 
     return ProbingResult();
@@ skipping 23 matching lines @@
                << " [cluster id: " << packet_info.probe_cluster_id
                << "] [send: " << cluster->size << " bytes / "
                << send_interval_ms << " ms = " << send_bps / 1000 << " kb/s]"
                << " [receive: " << cluster->size << " bytes / "
                << receive_interval_ms << " ms = " << receive_bps / 1000
                << " kb/s]";
 
   return ProbingResult(result_bps, packet_info.arrival_time_ms);
 }
 }  // namespace webrtc