Refactor BlimpConnection to TCPConnection

blimp/net/tcp_client_transport.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef BLIMP_NET_TCP_CLIENT_TRANSPORT_H_
 #define BLIMP_NET_TCP_CLIENT_TRANSPORT_H_
 
 #include <memory>
 #include <string>
 
@@ skipping 19 matching lines @@
 class BLIMP_NET_EXPORT TCPClientTransport : public BlimpTransport {
  public:
   TCPClientTransport(const net::IPEndPoint& ip_endpoint,
                      net::NetLog* net_log);
   ~TCPClientTransport() override;
 
   void SetClientSocketFactoryForTest(net::ClientSocketFactory* factory);
 
   // BlimpTransport implementation.
   void Connect(const net::CompletionCallback& callback) override;
-  std::unique_ptr<MessagePort> TakeMessagePort() override;
+  std::unique_ptr<BlimpConnection> MakeConnection() override;
+
+  std::unique_ptr<MessagePort> TakeMessagePort();

[Wez, 2016/11/09 22:47:17]

nit: Please add a comment to clarify why this is here.

+
   const char* GetName() const override;
 
  protected:
   // Called when the TCP connection completed.
   virtual void OnTCPConnectComplete(int result);
 
   // Called when the connection attempt completed or failed.
   // Resets |socket_| if |result| indicates a failure (!= net::OK).
   void OnConnectComplete(int result);
 
@@ skipping 11 matching lines @@
   net::CompletionCallback connect_callback_;
   net::ClientSocketFactory* socket_factory_ = nullptr;
   std::unique_ptr<net::StreamSocket> socket_;
 
   DISALLOW_COPY_AND_ASSIGN(TCPClientTransport);
 };
 
 }  // namespace blimp
 
 #endif  // BLIMP_NET_TCP_CLIENT_TRANSPORT_H_