Decouple DevTools from socket implementation.

net/base/tcp_listen_socket.h

Index: net/base/tcp_listen_socket.h
diff --git a/net/base/tcp_listen_socket.h b/net/base/tcp_listen_socket.h
index fed1df36689d7b2f67a8728a4af981b7ff3a614b..8476ab07814d79611ce4e25e9066d4cf6f480adf 100644
--- a/net/base/tcp_listen_socket.h
+++ b/net/base/tcp_listen_socket.h
@@ -39,6 +39,22 @@ class NET_EXPORT TCPListenSocket : public StreamListenSocket {
   DISALLOW_COPY_AND_ASSIGN(TCPListenSocket);
 };
 
+// Factory that can be used to instantiate TCPListenSocket.
+class TCPListenSocketFactory : public StreamListenSocketFactory {
+public:

[pfeldman, 2012/05/14 11:12:00]

space before public.

[Philippe, 2012/05/14 12:36:52]

Done.

+  TCPListenSocketFactory(const std::string& ip, int port);
+  virtual ~TCPListenSocketFactory();
+
+  // Returns a new instance of TCPListenSocket created through

[pfeldman, 2012/05/14 11:12:00]

Should be // StreamListenSocketFactory overrides

[Philippe, 2012/05/14 12:36:52]

Done.

+  // TCPListenSocket::CreateAndListen().
+  virtual scoped_refptr<StreamListenSocket> CreateAndListen(
+      StreamListenSocket::Delegate* delegate) OVERRIDE;
+
+private:
+  const std::string ip_;

[pfeldman, 2012/05/14 11:12:00]

DISALLOW_COPY_AND_ASSIGN is missing

[Philippe, 2012/05/14 12:36:52]

I added it although there is nothing here that prevents this class from being
copied.

[mmenke, 2012/05/14 20:45:19]

Copying classes with inheritance scares me.

[Philippe, 2012/05/15 14:39:09]

That's a good point.

+  const int port_;
+};
+
 }  // namespace net
 
 #endif  // NET_BASE_TCP_LISTEN_SOCKET_H_