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

Unified Diff: net/http/http_stream_factory_test_util.cc

Issue 2771263002: Retry upon 421 status code without IP pooling. (Closed)
Patch Set: Rebase. Created 3 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
Index: net/http/http_stream_factory_test_util.cc
diff --git a/net/http/http_stream_factory_test_util.cc b/net/http/http_stream_factory_test_util.cc
index 50d083073d7f4f9abbd980ca22d3fd906a8d01e1..55fb736e568db1042749cc56fb1a205976fb62f4 100644
--- a/net/http/http_stream_factory_test_util.cc
+++ b/net/http/http_stream_factory_test_util.cc
@@ -23,6 +23,7 @@ MockHttpStreamFactoryImplJob::MockHttpStreamFactoryImplJob(
const SSLConfig& proxy_ssl_config,
HostPortPair destination,
GURL origin_url,
+ bool enable_ip_based_pooling,
NetLog* net_log)
: HttpStreamFactoryImpl::Job(delegate,
job_type,
@@ -33,6 +34,7 @@ MockHttpStreamFactoryImplJob::MockHttpStreamFactoryImplJob(
proxy_ssl_config,
destination,
origin_url,
+ enable_ip_based_pooling,
net_log) {
DCHECK(!is_waiting());
}
@@ -49,6 +51,7 @@ MockHttpStreamFactoryImplJob::MockHttpStreamFactoryImplJob(
GURL origin_url,
AlternativeService alternative_service,
const ProxyServer& alternative_proxy_server,
+ bool enable_ip_based_pooling,
NetLog* net_log)
: HttpStreamFactoryImpl::Job(delegate,
job_type,
@@ -61,6 +64,7 @@ MockHttpStreamFactoryImplJob::MockHttpStreamFactoryImplJob(
origin_url,
alternative_service,
alternative_proxy_server,
+ enable_ip_based_pooling,
net_log) {}
MockHttpStreamFactoryImplJob::~MockHttpStreamFactoryImplJob() {}
@@ -82,6 +86,7 @@ HttpStreamFactoryImpl::Job* TestJobFactory::CreateJob(
const SSLConfig& proxy_ssl_config,
HostPortPair destination,
GURL origin_url,
+ bool enable_ip_based_pooling,
NetLog* net_log) {
DCHECK(!main_job_);
@@ -90,7 +95,7 @@ HttpStreamFactoryImpl::Job* TestJobFactory::CreateJob(
main_job_ = new MockHttpStreamFactoryImplJob(
delegate, job_type, session, request_info, priority, SSLConfig(),
- SSLConfig(), destination, origin_url, nullptr);
+ SSLConfig(), destination, origin_url, enable_ip_based_pooling, nullptr);
return main_job_;
}
@@ -106,12 +111,13 @@ HttpStreamFactoryImpl::Job* TestJobFactory::CreateJob(
HostPortPair destination,
GURL origin_url,
AlternativeService alternative_service,
+ bool enable_ip_based_pooling,
NetLog* net_log) {
DCHECK(!alternative_job_);
alternative_job_ = new MockHttpStreamFactoryImplJob(
delegate, job_type, session, request_info, priority, SSLConfig(),
SSLConfig(), destination, origin_url, alternative_service, ProxyServer(),
- nullptr);
+ enable_ip_based_pooling, nullptr);
return alternative_job_;
}
@@ -127,12 +133,13 @@ HttpStreamFactoryImpl::Job* TestJobFactory::CreateJob(
HostPortPair destination,
GURL origin_url,
const ProxyServer& alternative_proxy_server,
+ bool enable_ip_based_pooling,
NetLog* net_log) {
DCHECK(!alternative_job_);
alternative_job_ = new MockHttpStreamFactoryImplJob(
delegate, job_type, session, request_info, priority, SSLConfig(),
SSLConfig(), destination, origin_url, AlternativeService(),
- alternative_proxy_server, nullptr);
+ alternative_proxy_server, enable_ip_based_pooling, nullptr);
return alternative_job_;
}

Powered by Google App Engine
This is Rietveld 408576698