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 2c8fc625ef7eec1c7992f7f9b5c9135b45012c85..55fb736e568db1042749cc56fb1a205976fb62f4 100644 |
--- a/net/http/http_stream_factory_test_util.cc |
+++ b/net/http/http_stream_factory_test_util.cc |
@@ -4,8 +4,6 @@ |
#include "net/http/http_stream_factory_test_util.h" |
-#include <utility> |
- |
#include "net/proxy/proxy_info.h" |
using ::testing::_; |
@@ -21,7 +19,6 @@ MockHttpStreamFactoryImplJob::MockHttpStreamFactoryImplJob( |
HttpNetworkSession* session, |
const HttpRequestInfo& request_info, |
RequestPriority priority, |
- ProxyInfo proxy_info, |
const SSLConfig& server_ssl_config, |
const SSLConfig& proxy_ssl_config, |
HostPortPair destination, |
@@ -33,7 +30,6 @@ MockHttpStreamFactoryImplJob::MockHttpStreamFactoryImplJob( |
session, |
request_info, |
priority, |
- proxy_info, |
server_ssl_config, |
proxy_ssl_config, |
destination, |
@@ -49,7 +45,6 @@ MockHttpStreamFactoryImplJob::MockHttpStreamFactoryImplJob( |
HttpNetworkSession* session, |
const HttpRequestInfo& request_info, |
RequestPriority priority, |
- ProxyInfo proxy_info, |
const SSLConfig& server_ssl_config, |
const SSLConfig& proxy_ssl_config, |
HostPortPair destination, |
@@ -63,7 +58,6 @@ MockHttpStreamFactoryImplJob::MockHttpStreamFactoryImplJob( |
session, |
request_info, |
priority, |
- proxy_info, |
server_ssl_config, |
proxy_ssl_config, |
destination, |
@@ -82,40 +76,36 @@ TestJobFactory::TestJobFactory() |
TestJobFactory::~TestJobFactory() {} |
-std::unique_ptr<HttpStreamFactoryImpl::Job> TestJobFactory::CreateMainJob( |
+HttpStreamFactoryImpl::Job* TestJobFactory::CreateJob( |
HttpStreamFactoryImpl::Job::Delegate* delegate, |
HttpStreamFactoryImpl::JobType job_type, |
HttpNetworkSession* session, |
const HttpRequestInfo& request_info, |
RequestPriority priority, |
- const ProxyInfo& proxy_info, |
const SSLConfig& server_ssl_config, |
const SSLConfig& proxy_ssl_config, |
HostPortPair destination, |
GURL origin_url, |
bool enable_ip_based_pooling, |
NetLog* net_log) { |
+ DCHECK(!main_job_); |
+ |
if (override_main_job_url_) |
origin_url = main_job_alternative_url_; |
- auto main_job = base::MakeUnique<MockHttpStreamFactoryImplJob>( |
- delegate, job_type, session, request_info, priority, proxy_info, |
- SSLConfig(), SSLConfig(), destination, origin_url, |
- enable_ip_based_pooling, nullptr); |
- |
- // Keep raw pointer to Job but pass ownership. |
- main_job_ = main_job.get(); |
+ main_job_ = new MockHttpStreamFactoryImplJob( |
+ delegate, job_type, session, request_info, priority, SSLConfig(), |
+ SSLConfig(), destination, origin_url, enable_ip_based_pooling, nullptr); |
- return std::move(main_job); |
+ return main_job_; |
} |
-std::unique_ptr<HttpStreamFactoryImpl::Job> TestJobFactory::CreateAltSvcJob( |
+HttpStreamFactoryImpl::Job* TestJobFactory::CreateJob( |
HttpStreamFactoryImpl::Job::Delegate* delegate, |
HttpStreamFactoryImpl::JobType job_type, |
HttpNetworkSession* session, |
const HttpRequestInfo& request_info, |
RequestPriority priority, |
- const ProxyInfo& proxy_info, |
const SSLConfig& server_ssl_config, |
const SSLConfig& proxy_ssl_config, |
HostPortPair destination, |
@@ -123,24 +113,21 @@ std::unique_ptr<HttpStreamFactoryImpl::Job> TestJobFactory::CreateAltSvcJob( |
AlternativeService alternative_service, |
bool enable_ip_based_pooling, |
NetLog* net_log) { |
- auto alternative_job = base::MakeUnique<MockHttpStreamFactoryImplJob>( |
- delegate, job_type, session, request_info, priority, proxy_info, |
- SSLConfig(), SSLConfig(), destination, origin_url, alternative_service, |
- ProxyServer(), enable_ip_based_pooling, nullptr); |
- |
- // Keep raw pointer to Job but pass ownership. |
- alternative_job_ = alternative_job.get(); |
+ DCHECK(!alternative_job_); |
+ alternative_job_ = new MockHttpStreamFactoryImplJob( |
+ delegate, job_type, session, request_info, priority, SSLConfig(), |
+ SSLConfig(), destination, origin_url, alternative_service, ProxyServer(), |
+ enable_ip_based_pooling, nullptr); |
- return std::move(alternative_job); |
+ return alternative_job_; |
} |
-std::unique_ptr<HttpStreamFactoryImpl::Job> TestJobFactory::CreateAltProxyJob( |
+HttpStreamFactoryImpl::Job* TestJobFactory::CreateJob( |
HttpStreamFactoryImpl::Job::Delegate* delegate, |
HttpStreamFactoryImpl::JobType job_type, |
HttpNetworkSession* session, |
const HttpRequestInfo& request_info, |
RequestPriority priority, |
- const ProxyInfo& proxy_info, |
const SSLConfig& server_ssl_config, |
const SSLConfig& proxy_ssl_config, |
HostPortPair destination, |
@@ -148,15 +135,13 @@ std::unique_ptr<HttpStreamFactoryImpl::Job> TestJobFactory::CreateAltProxyJob( |
const ProxyServer& alternative_proxy_server, |
bool enable_ip_based_pooling, |
NetLog* net_log) { |
- auto alternative_job = base::MakeUnique<MockHttpStreamFactoryImplJob>( |
- delegate, job_type, session, request_info, priority, proxy_info, |
- SSLConfig(), SSLConfig(), destination, origin_url, AlternativeService(), |
+ DCHECK(!alternative_job_); |
+ alternative_job_ = new MockHttpStreamFactoryImplJob( |
+ delegate, job_type, session, request_info, priority, SSLConfig(), |
+ SSLConfig(), destination, origin_url, AlternativeService(), |
alternative_proxy_server, enable_ip_based_pooling, nullptr); |
- // Keep raw pointer to Job but pass ownership. |
- alternative_job_ = alternative_job.get(); |
- |
- return std::move(alternative_job); |
+ return alternative_job_; |
} |
} // namespace net |