Expose a static configuration value for the host to use for URLRequestTestHTTP tests.

net/url_request/url_request_test_util.cc

Index: net/url_request/url_request_test_util.cc
diff --git a/net/url_request/url_request_test_util.cc b/net/url_request/url_request_test_util.cc
index c296745969d1bb288c3cde713ac2bf6c9ca97439..206686489245660398dc4c23b37fcd219107c5bd 100644
--- a/net/url_request/url_request_test_util.cc
+++ b/net/url_request/url_request_test_util.cc
@@ -504,3 +504,18 @@ net::NetworkDelegate::AuthRequiredResponse TestNetworkDelegate::OnAuthRequired(
       kStageBeforeRedirect;  // a delegate can trigger a redirection
   return net::NetworkDelegate::AUTH_REQUIRED_RESPONSE_NO_ACTION;
 }
+
+namespace {
+char url_request_test_http_host_[256] = "127.0.0.1";

[eroman, 2012/02/16 02:41:00]

First off, please prefix globals with "g_".

Secondly, since this is test code, I think it is fine to have static
initializers. So in other words, you can get away with defining this global as a
std::string and not needing to do the fancy copying of C strings.

[erikwright (departed), 2012/02/16 16:04:13]

Done.

+}  // namespace
+
+void set_url_request_test_http_host(const std::string& host) {
+  strncpy(url_request_test_http_host_,
+         host.c_str(),
+         arraysize(url_request_test_http_host_) - 1);
+  url_request_test_http_host_[arraysize(url_request_test_http_host_) - 1] = 0;

[eroman, 2012/02/16 02:41:00]

nit: If we have a host literal string longer than 255 something is not right :-)
This is fine as-is, but you might alternately consider asserting the length is
smaller at the start.

[erikwright (departed), 2012/02/16 16:04:13]

N/A.

+}
+
+char* url_request_test_http_host() {

[eroman, 2012/02/16 02:41:00]

can you return a const char* so caller can't mutate the array?

[erikwright (departed), 2012/02/16 16:04:13]

Done.

+  return url_request_test_http_host_;
+}