Disable QUIC for 5 miuntes, subject to exponential backoff, when

net/quic/chromium/quic_stream_factory.h

 // 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.
 
 #ifndef NET_QUIC_QUIC_STREAM_FACTORY_H_
 #define NET_QUIC_QUIC_STREAM_FACTORY_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 445 matching lines @@
   // |peer_address| and |network|. If |network| is kInvalidNetworkHandle,
   // default network is used. If |packet| is not null, it is sent
   // on the new network, else a PING frame is sent.
   void MigrateSession(QuicChromiumClientSession* session,
                       IPEndPoint peer_address,
                       NetworkChangeNotifier::NetworkHandle network,
                       bool close_session_on_error,
                       const BoundNetLog& bound_net_log,
                       scoped_refptr<StringIOBuffer> packet);
 
+  // Called to re-enable QUIC when QUIC has been disabled.
+  void OpenFactory();
+  // If QUIC has been working well after having been recently
+  // disabled, clear the |consecutive_disabled_count_|.
+  void MaybeClearConsecutiveDisabledCount();
+
   bool require_confirmation_;
   NetLog* net_log_;
   HostResolver* host_resolver_;
   ClientSocketFactory* client_socket_factory_;
   HttpServerProperties* http_server_properties_;
   TransportSecurityState* transport_security_state_;
   CTVerifier* cert_transparency_verifier_;
   std::unique_ptr<QuicServerInfoFactory> quic_server_info_factory_;
   QuicCryptoClientStreamFactory* quic_crypto_client_stream_factory_;
   QuicRandom* random_generator_;
@@ skipping 66 matching lines @@
   // Set if we do not want to load server config from the disk cache.
   bool disable_disk_cache_;
 
   // Set if AES-GCM should be preferred, even if there is no hardware support.
   bool prefer_aes_;
 
   // True if QUIC should be disabled when there are timeouts with open
   // streams.
   bool disable_quic_on_timeout_with_open_streams_;
 
+  // Number of times in a row that QUIC has been disabled.
+  int consecutive_disabled_count_;
+  bool need_to_evaluate_consecutive_disabled_count_;

[ianswett, 2016/09/14 20:10:14]

Also, maybe just evaluate_consecutive_disabled_count_, or
use_consecutive_disabled_count_ would be easier to read?

After reading the code, I think this could be named better, but I don't have an
easy suggestion.  Conceptually, I'd consider the 5 minute timer time a
'probation' period of sorts where we're trying to figure out if QUIC works. 
arm_probation_timer_ is a bit confusing as well.

+
   // Size of the UDP receive buffer.
   int socket_receive_buffer_size_;
 
   // Set if we do want to delay TCP connection when it is racing with QUIC.
   bool delay_tcp_race_;
 
   // PING timeout for connections.
   QuicTime::Delta ping_timeout_;
   QuicTime::Delta reduced_ping_timeout_;
 
@@ skipping 55 matching lines @@
   const scoped_refptr<SSLConfigService> ssl_config_service_;
 
   base::WeakPtrFactory<QuicStreamFactory> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(QuicStreamFactory);
 };
 
 }  // namespace net
 
 #endif  // NET_QUIC_QUIC_STREAM_FACTORY_H_