Adds a method to TCPServerSocket to adopt a socket.

net/socket/socket_posix.cc

Index: net/socket/socket_posix.cc
diff --git a/net/socket/socket_posix.cc b/net/socket/socket_posix.cc
index 6955aa2a1e83a565c12f2700db5e2d1a618effce..a43f5176409055221b861aec629cfb39db9927c8 100644
--- a/net/socket/socket_posix.cc
+++ b/net/socket/socket_posix.cc
@@ -98,6 +98,16 @@ int SocketPosix::Open(int address_family) {
 
 int SocketPosix::AdoptConnectedSocket(SocketDescriptor socket,
                                       const SockaddrStorage& address) {
+  int rv = AdoptUnconnectedSocket(socket);
+  if (rv != OK) {
+    return rv;
+  }

[mmenke, 2017/04/21 19:20:38]

nit:  Don't use braces for one line if bodies.

[Raul Vera, 2017/05/01 05:35:25]

Done. But I'd like to point out that the style guide doesn't say anything about
this, so defaults back to the Google style guide, which says you can do it if
you prefer. I consider one-line if bodies without braces to be bugs waiting to
happen when someone adds a line and doesn't notice that the braces aren't there.
Go even eliminated braceless ifs for this reason, IIRC.

[mmenke, 2017/05/01 18:44:19]

Note that you should generally be consistent with code around you.  In this
case, net/ does not use braces in this case (Largely because Chromium style used
to say not to use braces for single line ifs).

+
+  SetPeerAddress(address);
+  return OK;
+}
+
+int SocketPosix::AdoptUnconnectedSocket(SocketDescriptor socket) {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK_EQ(kInvalidSocket, socket_fd_);
 
@@ -109,7 +119,6 @@ int SocketPosix::AdoptConnectedSocket(SocketDescriptor socket,
     return rv;
   }
 
-  SetPeerAddress(address);
   return OK;
 }