Attempt to fix flake in FullTabModeIE_TestPostReissue by avoiding loopback address.

chrome_frame/test/test_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 <windows.h>
 #include <objbase.h>
 #include <urlmon.h>
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/string_number_conversions.h"
 #include "base/string_piece.h"
 #include "base/string_util.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
+#include "chrome_frame/test/chrome_frame_test_utils.h"
 #include "chrome_frame/test/test_server.h"
 #include "net/base/tcp_listen_socket.h"
 #include "net/base/winsock_init.h"
 #include "net/http/http_util.h"
 
 namespace test_server {
 const char kDefaultHeaderTemplate[] =
     "HTTP/1.1 %hs\r\n"
     "Connection: close\r\n"
     "Content-Type: %hs\r\n"
@@ skipping 103 matching lines @@
   *headers = base::StringPrintf("HTTP/1.1 302 Found\r\n"
                                 "Connection: close\r\n"
                                 "Content-Length: 0\r\n"
                                 "Content-Type: text/html\r\n"
                                 "Location: %hs\r\n\r\n",
                                 redirect_url_.c_str());
   return true;
 }
 
 SimpleWebServer::SimpleWebServer(int port) {
-  CHECK(MessageLoop::current()) << "SimpleWebServer requires a message loop";
-  net::EnsureWinsockInit();
-  AddResponse(&quit_);
-  server_ = net::TCPListenSocket::CreateAndListen("127.0.0.1", port, this);
-  DCHECK(server_.get() != NULL);
+  Construct(chrome_frame_test::GetLocalIPv4Address(), port);
+}
+
+SimpleWebServer::SimpleWebServer(const std::string& address, int port) {
+  Construct(address, port);
 }
 
 SimpleWebServer::~SimpleWebServer() {
   ConnectionList::const_iterator it;
   for (it = connections_.begin(); it != connections_.end(); ++it)
     delete (*it);
   connections_.clear();
 }
 
+void SimpleWebServer::Construct(const std::string& address, int port) {
+  CHECK(MessageLoop::current()) << "SimpleWebServer requires a message loop";
+  net::EnsureWinsockInit();
+  AddResponse(&quit_);
+  host_ = address;
+  server_ = net::TCPListenSocket::CreateAndListen(address, port, this);
+  DCHECK(server_.get() != NULL);

[robertshield, 2012/08/22 20:18:27]

Since this is running in tests, can we log a failure message here as well?

[grt (UTC plus 2), 2012/08/24 15:36:40]

Done.

+}
+
 void SimpleWebServer::AddResponse(Response* response) {
   responses_.push_back(response);
 }
 
 void SimpleWebServer::DeleteAllResponses() {
   std::list<Response*>::const_iterator it;
   for (it = responses_.begin(); it != responses_.end(); ++it) {
     if ((*it) != &quit_)
       delete (*it);
   }
@@ skipping 236 matching lines @@
     data_.append(content_length_header);
     data_.append("\r\n");
   }
 
   MessageLoop::current()->PostDelayedTask(
       FROM_HERE, base::Bind(&ConfigurableConnection::SendChunk, this),
       base::TimeDelta::FromMilliseconds(options.timeout_));
 }
 
 }  // namespace test_server