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

Unified Diff: chrome/browser/io_thread.h

Issue 1505063002: QUIC - FieldTrial parameters not do PreConnect if QUIC can do 0RTT. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@disable_preconnect
Patch Set: rebase Created 5 years 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 | « no previous file | chrome/browser/io_thread.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/io_thread.h
diff --git a/chrome/browser/io_thread.h b/chrome/browser/io_thread.h
index f156ebd5b0247f646e28ea1b2c7e04476f27cddd..510bc59f23268000a20212c4968f7bed886ada7f 100644
--- a/chrome/browser/io_thread.h
+++ b/chrome/browser/io_thread.h
@@ -231,6 +231,7 @@ class IOThread : public content::BrowserThreadDelegate {
Optional<net::HostPortPair> origin_to_force_quic_on;
Optional<bool> quic_close_sessions_on_ip_change;
Optional<int> quic_idle_connection_timeout_seconds;
+ Optional<bool> quic_disable_preconnect_if_0rtt;
bool enable_user_alternate_protocol_ports;
// NetErrorTabHelper uses |dns_probe_service| to send DNS probes when a
// main frame load fails with a DNS error in order to provide more useful
@@ -445,6 +446,10 @@ class IOThread : public content::BrowserThreadDelegate {
static int GetQuicIdleConnectionTimeoutSeconds(
const VariationParameters& quic_trial_params);
+ // Returns true if PreConnect should be disabled if QUIC can do 0RTT.
+ static bool ShouldQuicDisablePreConnectIfZeroRtt(
+ const VariationParameters& quic_trial_params);
+
// Returns the maximum length for QUIC packets, based on any flags in
// |command_line| or the field trial. Returns 0 if there is an error
// parsing any of the options, or if the default value should be used.
« no previous file with comments | « no previous file | chrome/browser/io_thread.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698