Respect QuicAllowed policy for new streams

chrome/common/pref_names.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/common/pref_names.h"
 
 #include "base/macros.h"
 #include "build/build_config.h"
 #include "chrome/common/features.h"
 #include "chrome/common/pref_font_webkit_names.h"
@@ skipping 419 matching lines @@
 // next startup, based on what was actually needed during this startup.
 const char kDnsPrefetchingStartupList[] = "dns_prefetching.startup_list";
 
 // A list of host names used to fetch web pages, and their commonly used
 // sub-resource hostnames (and expected latency benefits from pre-resolving, or
 // preconnecting to, such sub-resource hostnames).
 // This list is adaptively grown and pruned.
 const char kDnsPrefetchingHostReferralList[] =
     "dns_prefetching.host_referral_list";
 
+// Controls if the QUIC protocol is allowed.
+const char kQuicAllowed[] = "quic.allowed";

[gab, 2016/12/08 20:23:27]

I don't see any other "quic." prefs, should this just go in net.?

e.g. "net.quic_allowed" ?

[mmenke, 2016/12/08 20:34:49]

Net doesn't do any command line parsing, to let it be configured on a
per-URLRequestContext basis, and net can't depend on prefs.

There are plenty of quic constants in hrome_switches.h.

[gab, 2016/12/08 21:49:45]

I meant, not in net/, but just have the pref be prefixed by "net." -- just like
the surrounding prefs here.

[pastarmovj, 2016/12/09 11:12:54]

+1 to a common prefix.

[pmarko, 2016/12/13 06:01:51]

Done.

+
 // Prefs for persisting HttpServerProperties.
 const char kHttpServerProperties[] = "net.http_server_properties";
 
 // Prefs for persisting network qualities.
 const char kNetworkQualities[] = "net.network_qualities";
 
 #if defined(OS_ANDROID)
 // Last time that a check for cloud policy management was done. This time is
 // recorded on Android so that retries aren't attempted on every startup.
 // Instead the cloud policy registration is retried at least 1 or 3 days later.
@@ skipping 1866 matching lines @@
 // How many times the search geolocation disclosure has been shown.
 const char kSearchGeolocationDisclosureShownCount[] =
     "search_geolocation_disclosure.shown_count";
 
 // When the disclosure was shown last.
 const char kSearchGeolocationDisclosureLastShowDate[] =
     "search_geolocation_disclosure.last_show_date";
 #endif
 
 }  // namespace prefs