Remove BaseChannel's dependency on TransportController.

webrtc/p2p/base/transportcontroller.h

 /*
  *  Copyright 2015 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 95 matching lines @@
   bool RemoveRemoteCandidates(const Candidates& candidates, std::string* err);
   bool ReadyForRemoteCandidates(const std::string& transport_name) const;
   // TODO(deadbeef): GetStats isn't const because all the way down to
   // OpenSSLStreamAdapter,
   // GetSslCipherSuite and GetDtlsSrtpCryptoSuite are not const. Fix this.
   bool GetStats(const std::string& transport_name, TransportStats* stats);
   void SetMetricsObserver(webrtc::MetricsObserverInterface* metrics_observer);
 
   // Creates a channel if it doesn't exist. Otherwise, increments a reference
   // count and returns an existing channel.
+  TransportChannel* CreateTransportChannel(const std::string& transport_name,

[Taylor Brandstetter, 2017/01/09 22:50:42]

If EnableBundle were called on the networking thread, we could avoid the
signaling thread versions of these methods.

[Zhi Huang, 2017/01/12 03:47:46]

This method could be called on other threads in WebRtcSession.

+                                           int component);
   virtual TransportChannel* CreateTransportChannel_n(
       const std::string& transport_name,
       int component);
 
   // Decrements a channel's reference count, and destroys the channel if
   // nothing is referencing it.
+  void DestroyTransportChannel(const std::string& transport_name,
+                               int component);
   virtual void DestroyTransportChannel_n(const std::string& transport_name,
                                          int component);
 
   void use_quic() { quic_ = true; }
   bool quic() const { return quic_; }
 
   // TODO(deadbeef): Remove all for_testing methods!
   const rtc::scoped_refptr<rtc::RTCCertificate>& certificate_for_testing()
       const {
     return certificate_;
@@ skipping 133 matching lines @@
   bool quic_ = false;
 
   webrtc::MetricsObserverInterface* metrics_observer_ = nullptr;
 
   RTC_DISALLOW_COPY_AND_ASSIGN(TransportController);
 };
 
 }  // namespace cricket
 
 #endif  // WEBRTC_P2P_BASE_TRANSPORTCONTROLLER_H_