Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(50)

Unified Diff: net/http/http_network_layer_unittest.cc

Issue 118100: Avoid doing concurrent DNS resolves of the same hostname (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Get compiling on mac Created 11 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/http/http_network_layer_unittest.cc
===================================================================
--- net/http/http_network_layer_unittest.cc (revision 18213)
+++ net/http/http_network_layer_unittest.cc (working copy)
@@ -26,15 +26,17 @@
};
TEST_F(HttpNetworkLayerTest, CreateAndDestroy) {
+ net::HostResolver host_resolver;
scoped_ptr<net::ProxyService> proxy_service(net::ProxyService::CreateNull());
- net::HttpNetworkLayer factory(proxy_service.get());
+ net::HttpNetworkLayer factory(&host_resolver, proxy_service.get());
scoped_ptr<net::HttpTransaction> trans(factory.CreateTransaction());
}
TEST_F(HttpNetworkLayerTest, Suspend) {
+ net::HostResolver host_resolver;
scoped_ptr<net::ProxyService> proxy_service(net::ProxyService::CreateNull());
- net::HttpNetworkLayer factory(proxy_service.get());
+ net::HttpNetworkLayer factory(&host_resolver, proxy_service.get());
scoped_ptr<net::HttpTransaction> trans(factory.CreateTransaction());
trans.reset();
@@ -50,8 +52,9 @@
}
TEST_F(HttpNetworkLayerTest, GoogleGET) {
+ net::HostResolver host_resolver;
scoped_ptr<net::ProxyService> proxy_service(net::ProxyService::CreateNull());
- net::HttpNetworkLayer factory(proxy_service.get());
+ net::HttpNetworkLayer factory(&host_resolver, proxy_service.get());
TestCompletionCallback callback;

Powered by Google App Engine
This is Rietveld 408576698