Support port range for IPC P2P UDP sockets.

content/browser/renderer_host/p2p/socket_host_tcp_server.cc

 // Copyright (c) 2012 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.
 
 #include "content/browser/renderer_host/p2p/socket_host_tcp_server.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/stl_util.h"
 #include "content/browser/renderer_host/p2p/socket_host_tcp.h"
@@ skipping 21 matching lines @@
 P2PSocketHostTcpServer::~P2PSocketHostTcpServer() {
   STLDeleteContainerPairSecondPointers(accepted_sockets_.begin(),
                                        accepted_sockets_.end());
 
   if (state_ == STATE_OPEN) {
     DCHECK(socket_.get());
     socket_.reset();
   }
 }
 
+// TODO(guidou): Add support for port range.

[tommi (sloooow) - chröme, 2016/07/11 17:54:59]

I guess this means a separate policy then?

[Guido Urdaneta, 2016/07/12 10:15:00]

In the meantime the only call site is
https://cs.chromium.org/chromium/src/content/renderer/p2p/ipc_socket_factory....
which ignores the range.

In this PS we pass the range to the browser, but ignore it there.
What do you think we should do in this CL?

[tommi (sloooow) - chröme, 2016/07/12 10:21:46]

I think it's OK to not address it in this CL but a followup CL would be good.

 bool P2PSocketHostTcpServer::Init(const net::IPEndPoint& local_address,
+                                  uint16_t min_port,
+                                  uint16_t max_port,
                                   const P2PHostAndIPEndPoint& remote_address) {
   DCHECK_EQ(state_, STATE_UNINITIALIZED);
 
   int result = socket_->Listen(local_address, kListenBacklog);
   if (result < 0) {
     LOG(ERROR) << "Listen() failed: " << result;
     OnError();
     return false;
   }
 
@@ skipping 92 matching lines @@
   return result.release();
 }
 
 bool P2PSocketHostTcpServer::SetOption(P2PSocketOption option,
                                        int value) {
   // Currently we don't have use case tcp server sockets are used for p2p.
   return false;
 }
 
 }  // namespace content