Remove usage of a deprecated TestServer constructor.

Remove usage of a deprecated TestServer constructor.

Hostname must now be explicitly specified (previously default was 127.0.0.1). See the following CL for further details:

http://codereview.chromium.org/9369029/

A follow-up CL will remove the deprecated constructor:

http://codereview.chromium.org/9431002/

BUG=114369
TEST=everything still compiles and passes


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=123486

[erikwright (departed), 2012-02-23 19:06:30]

Ping :)

(Mostly boiler-plate, though I gave in to an urge to remove duplicate code in
network_stats_unittest.cc . If that hinders the rest of the review I can undo
it.)

[eroman, 2012-02-23 20:47:11]

Was OOO yesterday. Looking now...

[eroman, 2012-02-23 21:06:19]

lgtm

http://codereview.chromium.org/9430050/diff/1/chrome/browser/net/network_stat...
File chrome/browser/net/network_stats_unittest.cc (right):

http://codereview.chromium.org/9430050/diff/1/chrome/browser/net/network_stat...
chrome/browser/net/network_stats_unittest.cc:17: template <typename T> class
NetworkStatsTest : public PlatformTest {
Could you propose this change separately? I would prefer this to be just the
refactor in the description.

http://codereview.chromium.org/9430050/diff/1/net/url_request/url_request_uni...
File net/url_request/url_request_unittest.cc (right):

http://codereview.chromium.org/9430050/diff/1/net/url_request/url_request_uni...
net/url_request/url_request_unittest.cc:3652: :
test_server_(TestServer::TYPE_FTP, TestServer::kLocalhost, FilePath()) {
I believe you want 4 spaces here.

[commit-bot: I haz the power, 2012-02-24 14:35:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/erikwright@chromium.org/9430050/9002

[commit-bot: I haz the power, 2012-02-24 16:16:22]

Change committed as 123486