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

Unified Diff: net/http/http_network_session.cc

Issue 2546533003: Respect QuicAllowed policy for new streams (Closed)
Patch Set: Reviewers' feedback Created 3 years, 12 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_network_session.cc
diff --git a/net/http/http_network_session.cc b/net/http/http_network_session.cc
index 3b2ca28d9fe3fb33557f429cd546c39354303c1d..49784e397cb15a08c505998057ff8ef1936fa5ad 100644
--- a/net/http/http_network_session.cc
+++ b/net/http/http_network_session.cc
@@ -73,19 +73,19 @@ const int32_t kSpdyStreamMaxRecvWindowSize = 6 * 1024 * 1024; // 6 MB
const int32_t kQuicSocketReceiveBufferSize = 1024 * 1024; // 1MB
HttpNetworkSession::Params::Params()
- : client_socket_factory(NULL),
- host_resolver(NULL),
- cert_verifier(NULL),
- channel_id_service(NULL),
- transport_security_state(NULL),
- cert_transparency_verifier(NULL),
- ct_policy_enforcer(NULL),
- proxy_service(NULL),
- ssl_config_service(NULL),
- http_auth_handler_factory(NULL),
- net_log(NULL),
- host_mapping_rules(NULL),
- socket_performance_watcher_factory(NULL),
+ : client_socket_factory(nullptr),
+ host_resolver(nullptr),
+ cert_verifier(nullptr),
+ channel_id_service(nullptr),
+ transport_security_state(nullptr),
+ cert_transparency_verifier(nullptr),
+ ct_policy_enforcer(nullptr),
+ proxy_service(nullptr),
+ ssl_config_service(nullptr),
+ http_auth_handler_factory(nullptr),
+ net_log(nullptr),
+ host_mapping_rules(nullptr),
+ socket_performance_watcher_factory(nullptr),
ignore_certificate_errors(false),
testing_fixed_http_port(0),
testing_fixed_https_port(0),
@@ -109,8 +109,8 @@ HttpNetworkSession::Params::Params()
quic_socket_receive_buffer_size(kQuicSocketReceiveBufferSize),
quic_delay_tcp_race(true),
quic_max_server_configs_stored_in_properties(0u),
- quic_clock(NULL),
- quic_random(NULL),
+ quic_clock(nullptr),
+ quic_random(nullptr),
quic_max_packet_length(kDefaultMaxPacketSize),
enable_user_alternate_protocol_ports(false),
quic_crypto_client_stream_factory(
@@ -128,7 +128,7 @@ HttpNetworkSession::Params::Params()
quic_force_hol_blocking(false),
quic_race_cert_verification(false),
quic_do_not_fragment(false),
- proxy_delegate(NULL),
+ proxy_delegate(nullptr),
enable_token_binding(false),
http_09_on_non_default_ports_enabled(false) {
quic_supported_versions.push_back(QUIC_VERSION_35);
@@ -292,7 +292,7 @@ std::unique_ptr<base::Value> HttpNetworkSession::SpdySessionPoolInfoToValue()
std::unique_ptr<base::Value> HttpNetworkSession::QuicInfoToValue() const {
std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
dict->Set("sessions", quic_stream_factory_.QuicStreamFactoryInfoToValue());
- dict->SetBoolean("quic_enabled", params_.enable_quic);
+ dict->SetBoolean("quic_enabled", IsQuicEnabled());
std::unique_ptr<base::ListValue> connection_options(new base::ListValue);
for (QuicTagVector::const_iterator it =
params_.quic_connection_options.begin();
@@ -358,7 +358,7 @@ bool HttpNetworkSession::IsProtocolEnabled(NextProto protocol) const {
case kProtoHTTP2:
return params_.enable_http2;
case kProtoQUIC:
- return params_.enable_quic;
+ return IsQuicEnabled();
}
NOTREACHED();
return false;
@@ -415,6 +415,14 @@ void HttpNetworkSession::DumpMemoryStats(
http_network_session_dump->guid());
}
+bool HttpNetworkSession::IsQuicEnabled() const {
+ return params_.enable_quic;
+}
+
+void HttpNetworkSession::SetQuicEnabled(bool quic_enabled) {
+ params_.enable_quic = quic_enabled;
+}
+
ClientSocketPoolManager* HttpNetworkSession::GetSocketPoolManager(
SocketPoolType pool_type) {
switch (pool_type) {

Powered by Google App Engine
This is Rietveld 408576698