Added TCPServerSocket.

net/socket/server_socket.h

Index: net/socket/server_socket.h
diff --git a/net/socket/server_socket.h b/net/socket/server_socket.h
new file mode 100644
index 0000000000000000000000000000000000000000..f037b036f413e10738fe5b19d0860af9ca4f755f
--- /dev/null
+++ b/net/socket/server_socket.h
@@ -0,0 +1,39 @@
+// Copyright (c) 2011 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 NET_SOCKET_SERVER_SOCKET_H_
+#define NET_SOCKET_SERVER_SOCKET_H_
+
+#include "base/scoped_ptr.h"
+#include "net/base/completion_callback.h"
+
+namespace net {
+
+class ClientSocket;
+class IPEndPoint;
+
+class ServerSocket {

[willchan no longer on Chromium, 2011/04/18 18:07:17]

Can you justify the need for this abstraction?

[Sergey Ulanov, 2011/04/18 19:40:28]

That's mainly to make it easier to unittest code that uses TCPServerSocket. I
could just make corresponding methods virtual in TCPSocketServer, but I think
having an interface is cleaner. I'll remove it if you disagree.

[willchan no longer on Chromium, 2011/04/18 20:08:56]

I see. I had thought you were planning on using this polymorphically in
production code. Coding to interfaces is good and makes testing easier, so I'm
fine with this as is.

+ public:
+  ServerSocket() { }
+  virtual ~ServerSocket() { }
+
+  // Bind the socket and start listening. Destroy the socket to stop
+  // listening.
+  virtual int Listen(const net::IPEndPoint& address, int backlog) = 0;
+
+  // Gets current address the socket is bound to.
+  virtual int GetLocalAddress(IPEndPoint* address) const = 0;
+
+  // Accept connection. Callback is called when new connection is
+  // accepted.
+  virtual int Accept(scoped_ptr<ClientSocket>* socket,
+                     CompletionCallback* callback) = 0;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ServerSocket);
+};
+
+}  // namespace net
+
+#endif  // NET_SOCKET_TCP_SERVER_SOCKET_H_