BitrateProber::CreateProbeCluster now only accept one parameter (bitrate_bps).

webrtc/modules/congestion_controller/probe_controller.cc

Index: webrtc/modules/congestion_controller/probe_controller.cc
diff --git a/webrtc/modules/congestion_controller/probe_controller.cc b/webrtc/modules/congestion_controller/probe_controller.cc
index 6ce7d6b5df37b0a9df2cbf5e38ca710029dcf1d1..6a5254a2817eeaa693d0e3dcd474f4ec29501e1d 100644
--- a/webrtc/modules/congestion_controller/probe_controller.cc
+++ b/webrtc/modules/congestion_controller/probe_controller.cc
@@ -20,12 +20,6 @@
 namespace webrtc {
 
 namespace {
-
-// Number of deltas between probes per cluster. On the very first cluster,
-// we will need kProbeDeltasPerCluster + 1 probes, but on a cluster following
-// another, we need kProbeDeltasPerCluster probes.
-constexpr int kProbeDeltasPerCluster = 5;
-
 // Maximum waiting time from the time of initiating probing to getting
 // the measured results back.
 constexpr int64_t kMaxWaitingTimeForProbingResultMs = 1000;
@@ -209,12 +203,10 @@ void ProbeController::InitiateProbing(
       probe_further = false;
     }
     if (first_cluster) {

[terelius, 2017/01/04 13:03:04]

Is this used? If not, please remove first_cluster.

[philipel, 2017/01/04 13:31:14]

Not used, removed.

-      pacer_->CreateProbeCluster(rtc::checked_cast<int>(bitrate),
-                                 kProbeDeltasPerCluster + 1);
+      pacer_->CreateProbeCluster(rtc::checked_cast<int>(bitrate));
       first_cluster = false;
     } else {
-      pacer_->CreateProbeCluster(rtc::checked_cast<int>(bitrate),
-                                 kProbeDeltasPerCluster);
+      pacer_->CreateProbeCluster(rtc::checked_cast<int>(bitrate));
     }
   }
   time_last_probing_initiated_ms_ = now_ms;