Chromium Code Reviews| Index: chrome/test/in_process_browser_test.cc |
| diff --git a/chrome/test/in_process_browser_test.cc b/chrome/test/in_process_browser_test.cc |
| index e3c064d3fa3904211dad96051cca0b117e8931f1..d97106efaa9db534e63ef270f3ccdce413cc4f45 100644 |
| --- a/chrome/test/in_process_browser_test.cc |
| +++ b/chrome/test/in_process_browser_test.cc |
| @@ -112,13 +112,15 @@ void InProcessBrowserTest::SetUp() { |
| params.ui_task = |
| NewRunnableMethod(this, &InProcessBrowserTest::RunTestOnMainThreadLoop); |
| - // TODO(sky): Don't make a real dns lookup here or simulate a failing |
| - // lookup. But if it's really needed then remove the TODO. |
| scoped_refptr<net::RuleBasedHostMapper> host_mapper( |
| new net::RuleBasedHostMapper()); |
| + // TODO(sky): Don't make a real dns lookup here or simulate a failing |
| + // lookup. |
| host_mapper->AllowDirectLookup("*.google.com"); |
| - net::ScopedHostMapper scoped_host_mapper; |
| - scoped_host_mapper.Init(host_mapper.get()); |
| + // See http://en.wikipedia.org/wiki/Web_Proxy_Autodiscovery_Protocol |
| + // We don't want to depend on it in test code. |
|
Finnur
2009/04/06 19:32:42
nit: depend? Is that the right word to use here?
|
| + host_mapper->AddSimulatedFailure("wpad"); |
| + net::ScopedHostMapper scoped_host_mapper(host_mapper.get()); |
| BrowserMain(params); |
| } |