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

Unified Diff: net/http/http_stream_factory_test_util.cc

Issue 2958133002: Change QuicStreamRequest::Request() to take a preferred QuicVersion so that (Closed)
Patch Set: Re #26 Created 3 years, 5 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_stream_factory_test_util.h ('k') | net/quic/chromium/quic_network_transaction_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 4d05c408d4ad4e143029caa498c4c41dfce0e041..eb45c36dc5d65582295e2d68dbde08e1f1e110a9 100644
--- a/net/http/http_stream_factory_test_util.cc
+++ b/net/http/http_stream_factory_test_util.cc
@@ -27,6 +27,7 @@ MockHttpStreamFactoryImplJob::MockHttpStreamFactoryImplJob(
HostPortPair destination,
GURL origin_url,
NextProto alternative_protocol,
+ QuicVersion quic_version,
const ProxyServer& alternative_proxy_server,
bool enable_ip_based_pooling,
NetLog* net_log)
@@ -41,6 +42,7 @@ MockHttpStreamFactoryImplJob::MockHttpStreamFactoryImplJob(
destination,
origin_url,
alternative_protocol,
+ quic_version,
alternative_proxy_server,
enable_ip_based_pooling,
net_log) {
@@ -75,7 +77,8 @@ std::unique_ptr<HttpStreamFactoryImpl::Job> TestJobFactory::CreateMainJob(
auto main_job = base::MakeUnique<MockHttpStreamFactoryImplJob>(
delegate, job_type, session, request_info, priority, proxy_info,
SSLConfig(), SSLConfig(), destination, origin_url, kProtoUnknown,
- ProxyServer(), enable_ip_based_pooling, net_log);
+ QUIC_VERSION_UNSUPPORTED, ProxyServer(), enable_ip_based_pooling,
+ net_log);
// Keep raw pointer to Job but pass ownership.
main_job_ = main_job.get();
@@ -95,12 +98,13 @@ std::unique_ptr<HttpStreamFactoryImpl::Job> TestJobFactory::CreateAltSvcJob(
HostPortPair destination,
GURL origin_url,
NextProto alternative_protocol,
+ QuicVersion quic_version,
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_protocol,
- ProxyServer(), enable_ip_based_pooling, net_log);
+ quic_version, ProxyServer(), enable_ip_based_pooling, net_log);
// Keep raw pointer to Job but pass ownership.
alternative_job_ = alternative_job.get();
@@ -125,7 +129,8 @@ std::unique_ptr<HttpStreamFactoryImpl::Job> TestJobFactory::CreateAltProxyJob(
auto alternative_job = base::MakeUnique<MockHttpStreamFactoryImplJob>(
delegate, job_type, session, request_info, priority, proxy_info,
SSLConfig(), SSLConfig(), destination, origin_url, kProtoUnknown,
- alternative_proxy_server, enable_ip_based_pooling, net_log);
+ QUIC_VERSION_UNSUPPORTED, alternative_proxy_server,
+ enable_ip_based_pooling, net_log);
// Keep raw pointer to Job but pass ownership.
alternative_job_ = alternative_job.get();
« no previous file with comments | « net/http/http_stream_factory_test_util.h ('k') | net/quic/chromium/quic_network_transaction_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698