Explicitly whitelist the test server port.

net/test/test_server.cc

 // Copyright (c) 2010 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 "net/test/test_server.h"
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
@@ skipping 140 matching lines @@
   if (!SetPythonPath())
     return false;
 
   if (!LaunchPython(testserver_path))
     return false;
 
   if (!WaitToStart()) {
     Stop();
     return false;
   }
 

[akalin, 2010/12/08 15:02:34]

Add a TODO(akalin) to add code to store all ports opened by the testserver and
to add them, so that we don't have to have code in other places (e.g.,
live_sync_test.cc) to do that.

[Bernhard Bauer, 2010/12/09 10:48:08]

Done.

+  net::explicitly_allowed_ports.insert(host_port_pair_.port());
+
   started_ = true;
   return true;
 }
 
 bool TestServer::Stop() {
   if (!process_handle_)
     return true;
 
   started_ = false;
 
   // First check if the process has already terminated.
   bool ret = base::WaitForSingleProcess(process_handle_, 0);
   if (!ret)
     ret = base::KillProcess(process_handle_, 1, true);
 
   if (ret) {
     base::CloseProcessHandle(process_handle_);
     process_handle_ = base::kNullProcessHandle;
   } else {
     VLOG(1) << "Kill failed?";
   }
 
+  net::explicitly_allowed_ports.erase(host_port_pair_.port());
+
   return ret;
 }
 
 const HostPortPair& TestServer::host_port_pair() const {
   DCHECK(started_);
   return host_port_pair_;
 }
 
 const DictionaryValue& TestServer::server_data() const {
   DCHECK(started_);
@@ skipping 210 matching lines @@
   }
   if ((port <= 0) || (port > kuint16max)) {
     LOG(ERROR) << "Invalid port value: " << port;
     return false;
   }
   host_port_pair_.set_port(port);
   return true;
 }
 
 }  // namespace net