Move congestion controller processing to the pacer thread.

webrtc/modules/congestion_controller/congestion_controller.cc

 /*
  *  Copyright (c) 2012 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 309 matching lines @@
   transport_feedback_adapter_.OnSentPacket(sent_packet.packet_id,
                                            sent_packet.send_time_ms);
 }
 
 void CongestionController::OnRttUpdate(int64_t avg_rtt_ms, int64_t max_rtt_ms) {
   remote_bitrate_estimator_->OnRttUpdate(avg_rtt_ms, max_rtt_ms);
   transport_feedback_adapter_.OnRttUpdate(avg_rtt_ms, max_rtt_ms);
 }
 
 int64_t CongestionController::TimeUntilNextProcess() {
-  return std::min(bitrate_controller_->TimeUntilNextProcess(),
-                  remote_bitrate_estimator_->TimeUntilNextProcess());
+  // TODO(nisse): Any prettier way to get the minimum of 4 values?
+#if 0
+  return std::min(std::min(bitrate_controller_->TimeUntilNextProcess(),
+                           remote_bitrate_estimator_->TimeUntilNextProcess()),
+                  std::min(pacer_->TimeUntilNextProcess(),
+                           remote_estimator_proxy_.TimeUntilNextProcess()));
+#else
+  // TODO(nisse): I think this is C++11. Ok?

[stefan-webrtc, 2017/01/16 15:42:46]

I think this should be fine

[nisse-webrtc, 2017/01/16 16:14:03]

Nice, doing it this way then.

+  return std::min({bitrate_controller_->TimeUntilNextProcess(),
+                   remote_bitrate_estimator_->TimeUntilNextProcess(),
+                   pacer_->TimeUntilNextProcess(),
+                   remote_estimator_proxy_.TimeUntilNextProcess()});
+#endif
 }
 
 void CongestionController::Process() {
   bitrate_controller_->Process();
   remote_bitrate_estimator_->Process();
   probe_controller_->Process();
+  pacer_->Process();
+  remote_estimator_proxy_.Process();
   MaybeTriggerOnNetworkChanged();
 }
 
 void CongestionController::MaybeTriggerOnNetworkChanged() {
   // TODO(perkj): |observer_| can be nullptr if the ctor that accepts a
   // BitrateObserver is used. Remove this check once the ctor is removed.
   if (!observer_)
     return;
 
   uint32_t bitrate_bps;
@@ skipping 39 matching lines @@
 bool CongestionController::IsSendQueueFull() const {
   return pacer_->ExpectedQueueTimeMs() > PacedSender::kMaxQueueLengthMs;
 }
 
 bool CongestionController::IsNetworkDown() const {
   rtc::CritScope cs(&critsect_);
   return network_state_ == kNetworkDown;
 }
 
 }  // namespace webrtc