Extract Proxy Resolution out of HttpStreamFactoryImpl::Job

net/http/http_stream_factory_impl_request_unittest.cc

Index: net/http/http_stream_factory_impl_request_unittest.cc
diff --git a/net/http/http_stream_factory_impl_request_unittest.cc b/net/http/http_stream_factory_impl_request_unittest.cc
index 167dd87181f0ce645be4b77eacafec7b2557d261..b5df9271ff4fc5be0115c06eecc13403ea0e7aa0 100644
--- a/net/http/http_stream_factory_impl_request_unittest.cc
+++ b/net/http/http_stream_factory_impl_request_unittest.cc
@@ -25,7 +25,13 @@ class HttpStreamFactoryImplRequestTest : public ::testing::Test {};
 
 // Make sure that Request passes on its priority updates to its jobs.
 TEST_F(HttpStreamFactoryImplRequestTest, SetPriority) {
+  SequencedSocketData data(nullptr, 0, nullptr, 0);
+  data.set_connect_data(MockConnect(ASYNC, OK));
+  auto ssl_data = base::MakeUnique<SSLSocketDataProvider>(ASYNC, OK);
   SpdySessionDependencies session_deps(ProxyService::CreateDirect());
+  session_deps.socket_factory->AddSocketDataProvider(&data);
+  session_deps.socket_factory->AddSSLSocketDataProvider(ssl_data.get());
+
   std::unique_ptr<HttpNetworkSession> session =
       SpdySessionDependencies::SpdyCreateSession(&session_deps);
   HttpStreamFactoryImpl* factory =
@@ -33,20 +39,20 @@ TEST_F(HttpStreamFactoryImplRequestTest, SetPriority) {
   MockHttpStreamRequestDelegate request_delegate;
   TestJobFactory job_factory;
   HttpRequestInfo request_info;
+  request_info.url = GURL("http://www.example.com/");
   auto job_controller = base::MakeUnique<HttpStreamFactoryImpl::JobController>(
       factory, &request_delegate, session.get(), &job_factory, request_info,
       /* is_preconnect = */ false,
       /* enable_ip_based_pooling = */ true,
-      /* enable_alternative_services = */ true);
+      /* enable_alternative_services = */ true, SSLConfig(), SSLConfig());
   HttpStreamFactoryImpl::JobController* job_controller_raw_ptr =
       job_controller.get();
   factory->job_controller_set_.insert(std::move(job_controller));
 
   std::unique_ptr<HttpStreamFactoryImpl::Request> request(
       job_controller_raw_ptr->Start(
-          request_info, &request_delegate, nullptr, NetLogWithSource(),
-          HttpStreamRequest::HTTP_STREAM, DEFAULT_PRIORITY, SSLConfig(),
-          SSLConfig()));
+          &request_delegate, nullptr, NetLogWithSource(),
+          HttpStreamRequest::HTTP_STREAM, DEFAULT_PRIORITY));
   EXPECT_TRUE(job_controller_raw_ptr->main_job());
   EXPECT_EQ(DEFAULT_PRIORITY, job_controller_raw_ptr->main_job()->priority());
 
@@ -59,5 +65,7 @@ TEST_F(HttpStreamFactoryImplRequestTest, SetPriority) {
 
   request->SetPriority(IDLE);
   EXPECT_EQ(IDLE, job_controller_raw_ptr->main_job()->priority());
+  EXPECT_TRUE(data.AllReadDataConsumed());
+  EXPECT_TRUE(data.AllWriteDataConsumed());
 }
 }  // namespace net