Start ICE connectivity checks as soon as the first pair is pingable.

webrtc/p2p/base/transport.cc

 /*
  *  Copyright 2004 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 96 matching lines @@
   if (!ret) {
     return false;
   }
 
   // If PRANSWER/ANSWER is set, we should decide transport protocol type.
   if (action == CA_PRANSWER || action == CA_ANSWER) {
     ret &= NegotiateTransportDescription(action, error_desc);
   }
   if (ret) {
     local_description_set_ = true;
-    ConnectChannels();
   }
 
   return ret;
 }
 
 bool Transport::SetRemoteTransportDescription(
     const TransportDescription& description,
     ContentAction action,
     std::string* error_desc) {
   bool ret = true;
@@ skipping 46 matching lines @@
   channel->SetIceConfig(ice_config_);
   // TODO(ronghuawu): Change CreateChannel to be able to return error since
   // below Apply**Description calls can fail.
   if (local_description_)
     ApplyLocalTransportDescription(channel, nullptr);
   if (remote_description_)
     ApplyRemoteTransportDescription(channel, nullptr);
   if (local_description_ && remote_description_)
     ApplyNegotiatedTransportDescription(channel, nullptr);
 
-  if (connect_requested_) {
-    channel->Connect();
-  }
   return channel;
 }
 
 TransportChannelImpl* Transport::GetChannel(int component) {
   auto iter = channels_.find(component);
   return (iter != channels_.end()) ? iter->second : nullptr;
 }
 
 bool Transport::HasChannels() {
   return !channels_.empty();
 }
 
 void Transport::DestroyChannel(int component) {
   auto iter = channels_.find(component);
   if (iter == channels_.end())
     return;
 
   TransportChannelImpl* channel = iter->second;
   channels_.erase(iter);
   DestroyTransportChannel(channel);
 }
 
-void Transport::ConnectChannels() {
-  if (connect_requested_ || channels_.empty())
-    return;
-
-  connect_requested_ = true;
-
-  RTC_DCHECK(local_description_);
-
-  CallChannels(&TransportChannelImpl::Connect);
-}
-
 void Transport::MaybeStartGathering() {
-  if (connect_requested_) {
-    CallChannels(&TransportChannelImpl::MaybeStartGathering);
-  }
+  CallChannels(&TransportChannelImpl::MaybeStartGathering);
 }
 
 void Transport::DestroyAllChannels() {
   for (const auto& kv : channels_) {
     DestroyTransportChannel(kv.second);
   }
   channels_.clear();
   channels_destroyed_ = true;
 }
 
@@ skipping 248 matching lines @@
     }
 
     // If local is passive, local will act as server.
   }
 
   *ssl_role = is_remote_server ? rtc::SSL_CLIENT : rtc::SSL_SERVER;
   return true;
 }
 
 }  // namespace cricket