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

Unified Diff: net/http/http_proxy_client_socket_pool_unittest.cc

Issue 1779733003: Fix bug in net::RequestPriority -> HTTP/2 dependency conversion. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Make bi-directional stream unittests enable priority->dependency setting. Created 4 years, 9 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/http/http_network_transaction_unittest.cc ('k') | net/net.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_proxy_client_socket_pool_unittest.cc
diff --git a/net/http/http_proxy_client_socket_pool_unittest.cc b/net/http/http_proxy_client_socket_pool_unittest.cc
index 29c10d0679e3d81f5faa3206b66ae750bc959043..7fea5f1fdf74ec513cc8c0c4bdf381b0b8df6a36 100644
--- a/net/http/http_proxy_client_socket_pool_unittest.cc
+++ b/net/http/http_proxy_client_socket_pool_unittest.cc
@@ -86,20 +86,18 @@ class HttpProxyClientSocketPoolTest
NULL,
session_deps_.ssl_config_service.get(),
BoundNetLog().net_log()),
- session_(CreateNetworkSession()),
spdy_util_(GetParam().protocol, GetParam().priority_to_dependency),
pool_(kMaxSockets,
kMaxSocketsPerGroup,
&transport_socket_pool_,
&ssl_socket_pool_,
NULL) {
- SpdySession::SetPriorityDependencyDefaultForTesting(
- GetParam().priority_to_dependency);
+ session_deps_.enable_priority_dependencies =
+ GetParam().priority_to_dependency;
+ session_ = CreateNetworkSession();
}
- virtual ~HttpProxyClientSocketPoolTest() {
- SpdySession::SetPriorityDependencyDefaultForTesting(false);
- }
+ virtual ~HttpProxyClientSocketPoolTest() {}
void AddAuthToCache() {
const base::string16 kFoo(base::ASCIIToUTF16("foo"));
@@ -219,7 +217,7 @@ class HttpProxyClientSocketPoolTest
scoped_ptr<CertVerifier> cert_verifier_;
SSLClientSocketPool ssl_socket_pool_;
- const scoped_ptr<HttpNetworkSession> session_;
+ scoped_ptr<HttpNetworkSession> session_;
protected:
SpdyTestUtil spdy_util_;
« no previous file with comments | « net/http/http_network_transaction_unittest.cc ('k') | net/net.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698