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

Unified Diff: net/quic/chromium/quic_stream_factory.h

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/quic/chromium/quic_network_transaction_unittest.cc ('k') | net/quic/chromium/quic_stream_factory.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/chromium/quic_stream_factory.h
diff --git a/net/quic/chromium/quic_stream_factory.h b/net/quic/chromium/quic_stream_factory.h
index 3942735a80d4cefad0da85a92d3d4feeb4e9e5ff..81b7841f82c213208fc766665d81272ce8883fec 100644
--- a/net/quic/chromium/quic_stream_factory.h
+++ b/net/quic/chromium/quic_stream_factory.h
@@ -115,6 +115,7 @@ class NET_EXPORT_PRIVATE QuicStreamRequest {
// |destination| will be resolved and resulting IPEndPoint used to open a
// QuicConnection. This can be different than HostPortPair::FromURL(url).
int Request(const HostPortPair& destination,
+ QuicVersion quic_version,
PrivacyMode privacy_mode,
int cert_verify_flags,
const GURL& url,
@@ -202,7 +203,6 @@ class NET_EXPORT_PRIVATE QuicStreamFactory
QuicClock* clock,
size_t max_packet_length,
const std::string& user_agent_id,
- const QuicVersionVector& supported_versions,
bool store_server_configs_in_properties,
bool close_sessions_on_ip_change,
bool mark_quic_broken_when_network_blackholes,
@@ -233,6 +233,7 @@ class NET_EXPORT_PRIVATE QuicStreamFactory
// OnRequestComplete asynchronously.
int Create(const QuicServerId& server_id,
const HostPortPair& destination,
+ QuicVersion quic_version,
int cert_verify_flags,
const GURL& url,
QuicStringPiece method,
@@ -404,6 +405,7 @@ class NET_EXPORT_PRIVATE QuicStreamFactory
bool HasActiveJob(const QuicServerId& server_id) const;
bool HasActiveCertVerifierJob(const QuicServerId& server_id) const;
int CreateSession(const QuicSessionKey& key,
+ const QuicVersion& quic_version,
int cert_verify_flags,
bool require_confirmation,
const AddressList& address_list,
@@ -515,8 +517,6 @@ class NET_EXPORT_PRIVATE QuicStreamFactory
// Map of QuicServerId to owning CertVerifierJob.
CertVerifierJobMap active_cert_verifier_jobs_;
- const QuicVersionVector supported_versions_;
-
// True if QUIC should be marked as broken when a connection blackholes after
// the handshake is confirmed.
bool mark_quic_broken_when_network_blackholes_;
« no previous file with comments | « net/quic/chromium/quic_network_transaction_unittest.cc ('k') | net/quic/chromium/quic_stream_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698