Nuke TickTime::UseFakeClock.

webrtc/call/congestion_controller.cc

Index: webrtc/call/congestion_controller.cc
diff --git a/webrtc/call/congestion_controller.cc b/webrtc/call/congestion_controller.cc
index 18c880cfc2e6621548d81e2de2dbf5d9f3ab671f..4d346845c044ccdaee2a053da7fc89523d5d2722 100644
--- a/webrtc/call/congestion_controller.cc
+++ b/webrtc/call/congestion_controller.cc
@@ -147,19 +147,19 @@ class WrappingBitrateEstimator : public RemoteBitrateEstimator {
 CongestionController::CongestionController(ProcessThread* process_thread,
                                            CallStats* call_stats,
                                            BitrateObserver* bitrate_observer)
-    : remb_(new VieRemb()),
+    : clock_(Clock::GetRealTimeClock()),

[mflodman, 2015/12/09 10:11:25]

We might as well skip this member, inject an instance here and not have a
pointer in this class, it;s not used anywhere but in the constructor.

[pbos-webrtc, 2015/12/09 10:19:04]

Done.

+      remb_(new VieRemb(clock_)),
       packet_router_(new PacketRouter()),
-      pacer_(new PacedSender(Clock::GetRealTimeClock(),
+      pacer_(new PacedSender(clock_,
                              packet_router_.get(),
                              BitrateController::kDefaultStartBitrateKbps,
                              PacedSender::kDefaultPaceMultiplier *
                                  BitrateController::kDefaultStartBitrateKbps,
                              0)),
       remote_bitrate_estimator_(
-          new WrappingBitrateEstimator(remb_.get(), Clock::GetRealTimeClock())),
+          new WrappingBitrateEstimator(remb_.get(), clock_)),
       remote_estimator_proxy_(
-          new RemoteEstimatorProxy(Clock::GetRealTimeClock(),
-                                   packet_router_.get())),
+          new RemoteEstimatorProxy(clock_, packet_router_.get())),
       process_thread_(process_thread),
       call_stats_(call_stats),
       pacer_thread_(ProcessThread::Create("PacerThread")),