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

Unified Diff: net/url_request/url_request_unittest.cc

Issue 1298253002: Remove reference counting from HttpNetworkSession. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: test Created 5 years, 2 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
« no previous file with comments | « net/url_request/url_request_test_util.cc ('k') | net/url_request/view_cache_helper_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/url_request/url_request_unittest.cc
diff --git a/net/url_request/url_request_unittest.cc b/net/url_request/url_request_unittest.cc
index 87ad8ebe878e645cf7b60725e11b6e8014e1a900..a3d7325ba73a50b5eded960d95ec565a96bb0f20 100644
--- a/net/url_request/url_request_unittest.cc
+++ b/net/url_request/url_request_unittest.cc
@@ -7081,12 +7081,14 @@ TEST_F(URLRequestTestHTTP, NetworkSuspendTest) {
default_context_.http_auth_handler_factory();
params.network_delegate = &default_network_delegate_;
params.http_server_properties = default_context_.http_server_properties();
+ scoped_ptr<HttpNetworkSession> network_session(
+ new HttpNetworkSession(params));
scoped_ptr<HttpNetworkLayer> network_layer(
- new HttpNetworkLayer(new HttpNetworkSession(params)));
+ new HttpNetworkLayer(network_session.get()));
network_layer->OnSuspend();
HttpCache http_cache(network_layer.release(), default_context_.net_log(),
- HttpCache::DefaultBackend::InMemory(0));
+ HttpCache::DefaultBackend::InMemory(0), true);
TestURLRequestContext context(true);
context.set_http_transaction_factory(&http_cache);
@@ -8149,9 +8151,10 @@ TEST_F(HTTPSRequestTest, SSLSessionCacheShardTest) {
params.http_server_properties = default_context_.http_server_properties();
params.ssl_session_cache_shard = "alternate";
+ scoped_ptr<HttpNetworkSession> network_session(
+ new HttpNetworkSession(params));
scoped_ptr<HttpCache> cache(new HttpCache(
- new HttpNetworkSession(params),
- HttpCache::DefaultBackend::InMemory(0)));
+ network_session.get(), HttpCache::DefaultBackend::InMemory(0), false));
default_context_.set_http_transaction_factory(cache.get());
« no previous file with comments | « net/url_request/url_request_test_util.cc ('k') | net/url_request/view_cache_helper_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698