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

Unified Diff: net/http/http_stream_factory_impl_job_controller_unittest.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_impl_job_controller_unittest.cc
diff --git a/net/http/http_stream_factory_impl_job_controller_unittest.cc b/net/http/http_stream_factory_impl_job_controller_unittest.cc
index b3b016c5def011a66396b15365c6905e9888814b..6d95f0f1f8747a28769df89f1cce31015f99a7fc 100644
--- a/net/http/http_stream_factory_impl_job_controller_unittest.cc
+++ b/net/http/http_stream_factory_impl_job_controller_unittest.cc
@@ -162,7 +162,7 @@ class HttpStreamFactoryImplJobControllerTest : public ::testing::Test {
static_cast<HttpStreamFactoryImpl*>(session_->http_stream_factory());
job_controller_ = new HttpStreamFactoryImpl::JobController(
factory_, &request_delegate_, session_.get(), &job_factory_,
- request_info, is_preconnect_);
+ request_info, is_preconnect_, /* enable_ip_based_pooling = */ true);
HttpStreamFactoryImplPeer::AddJobController(factory_, job_controller_);
}
@@ -203,6 +203,7 @@ class HttpStreamFactoryImplJobControllerTest : public ::testing::Test {
private:
bool use_alternative_proxy_;
bool is_preconnect_;
+
// Not owned by |this|.
TestProxyDelegate* test_proxy_delegate_;
@@ -1336,7 +1337,8 @@ class HttpStreamFactoryImplJobControllerPreconnectTest
request_info_.url = GURL("https://www.example.com");
job_controller_ = new HttpStreamFactoryImpl::JobController(
factory_, &request_delegate_, session_.get(), &job_factory_,
- request_info_, true);
+ request_info_, /* is_preconnect = */ true,
+ /* enable_ip_based_pooling = */ true);
HttpStreamFactoryImplPeer::AddJobController(factory_, job_controller_);
}

Powered by Google App Engine
This is Rietveld 408576698