Start probes only after network is connected.

webrtc/call/call.cc

Index: webrtc/call/call.cc
diff --git a/webrtc/call/call.cc b/webrtc/call/call.cc
index d7272170841c665883fff1e4ceae0adb1e12d66b..1f89b1efb85bbe7f11d94addc67766c0231886d2 100644
--- a/webrtc/call/call.cc
+++ b/webrtc/call/call.cc
@@ -249,8 +249,8 @@ Call::Call(const Call::Config& config)
       call_stats_(new CallStats(clock_)),
       bitrate_allocator_(new BitrateAllocator(this)),
       config_(config),
-      audio_network_state_(kNetworkUp),
-      video_network_state_(kNetworkUp),
+      audio_network_state_(kNetworkDown),
+      video_network_state_(kNetworkDown),
       receive_crit_(RWLockWrapper::CreateRWLock()),
       send_crit_(RWLockWrapper::CreateRWLock()),
       event_log_(config.event_log),
@@ -281,6 +281,7 @@ Call::Call(const Call::Config& config)
   Trace::CreateTrace();
   call_stats_->RegisterStatsObserver(congestion_controller_.get());
 
+  congestion_controller_->SignalNetworkState(kNetworkDown);

[honghaiz3, 2016/11/07 23:31:41]

The initial values of network state in different classes (here,
CongestionController, and ProbeController) are different. Does it make sense to
set them all to be kNetworkDown and avoid this call here?

[Sergey Ulanov, 2016/11/08 00:10:57]

There problem is that that CongestionController is used elsewhere without
calling SignalNetworkState. Changing CongesionController to be in Down state by
default would break that logic.

   congestion_controller_->SetBweBitrates(
       config_.bitrate_config.min_bitrate_bps,
       config_.bitrate_config.start_bitrate_bps,