Adding OrtcFactory, and changing UdpTransport to match current plan.

webrtc/p2p/base/udptransportchannel.h

Index: webrtc/p2p/base/udptransportchannel.h
diff --git a/webrtc/p2p/base/udptransportchannel.h b/webrtc/p2p/base/udptransportchannel.h
index 2152fb7308046cd21f31e30947bd2d407d93687c..06d63b219c9b6afc8950058986fab6bddb277ff2 100644
--- a/webrtc/p2p/base/udptransportchannel.h
+++ b/webrtc/p2p/base/udptransportchannel.h
@@ -14,27 +14,38 @@
 #include <memory>
 #include <string>
 
+#include "webrtc/api/udptransportinterface.h"
+#include "webrtc/base/asyncpacketsocket.h"  // For PacketOptions.
 #include "webrtc/base/optional.h"
 #include "webrtc/base/thread_checker.h"
 #include "webrtc/p2p/base/packettransportinterface.h"
 
 namespace rtc {
 class AsyncPacketSocket;
-class PhysicalSocketServer;
+struct PacketTime;
+struct SentPacket;
 class SocketAddress;
-class SocketServer;
-class Thread;
 }
 
 namespace cricket {
 
-class UdpTransportChannel : public rtc::PacketTransportInterface {
+// Implementation of UdpTransportInterface.
+// Used by OrtcFactory.
+class UdpTransport : public webrtc::UdpTransportInterface,
+                     public rtc::PacketTransportInterface {
  public:
-  enum class State { INIT, CONNECTING, CONNECTED, FAILED };
-  explicit UdpTransportChannel(const std::string& transport_name);
-  UdpTransportChannel(const std::string& transport_name, rtc::SocketServer* ss);
-  ~UdpTransportChannel();
-
+  // |transport_name| is only used for identification/logging.
+  // |socket| must be non-null.
+  UdpTransport(const std::string& transport_name,
+               std::unique_ptr<rtc::AsyncPacketSocket> socket);
+  ~UdpTransport();
+
+  // Overrides of UdpTransportInterface, used by the API consumer.
+  rtc::SocketAddress GetLocalAddress() const override;
+  bool SetRemoteAddress(const rtc::SocketAddress& addr) override;
+  rtc::SocketAddress GetRemoteAddress() const override;
+
+  // Overrides of PacketTransportInterface, used by webrtc internally.
   const std::string debug_name() const override { return transport_name_; }
 
   bool receiving() const override {
@@ -53,24 +64,6 @@ class UdpTransportChannel : public rtc::PacketTransportInterface {
 
   int GetError() override { return send_error_; }
 
-  State state() const {
-    RTC_DCHECK_RUN_ON(&network_thread_checker_);
-    return state_;
-  }
-
-  // Start() makes UdpTransportChannel transition from state INIT to CONNECTING.
-  // It creates the local UDP socket and binds it to a port.
-  // Consider checking state() after calling Start().
-  void Start();
-
-  void SetRemoteParameters(const rtc::SocketAddress& remote);
-
-  // Returned optional does not have a value if in the INIT or FAILED state.
-  // Consider checking state() before calling local_parameters().
-  rtc::Optional<rtc::SocketAddress> local_parameters() const {
-    return local_parameters_;
-  }
-
  private:
   void OnSocketReadPacket(rtc::AsyncPacketSocket* socket,
                           const char* data,
@@ -79,16 +72,12 @@ class UdpTransportChannel : public rtc::PacketTransportInterface {
                           const rtc::PacketTime& packet_time);
   void OnSocketSentPacket(rtc::AsyncPacketSocket* socket,
                           const rtc::SentPacket& packet);
-  void SetState(State state);  // Set State and Signal.
   bool IsLocalConsistent();
-  void UpdateState();
   std::string transport_name_;
-  rtc::SocketServer* socket_server_;
-  State state_ = State::INIT;
   int send_error_ = 0;
   std::unique_ptr<rtc::AsyncPacketSocket> socket_;
-  rtc::Optional<rtc::SocketAddress> local_parameters_;
-  rtc::Optional<rtc::SocketAddress> remote_parameters_;
+  // If not set, will be an "nil" address ("IsNil" returns true).
+  rtc::SocketAddress remote_address_;
   rtc::ThreadChecker network_thread_checker_;
 };
 }  // namespace cricket