Mark QUIC broken when the network blackholes after the handshake

net/quic/chromium/quic_http_stream.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_CHROMIUM_QUIC_HTTP_STREAM_H_
 #define NET_QUIC_CHROMIUM_QUIC_HTTP_STREAM_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 24 matching lines @@
 // The QuicHttpStream is a QUIC-specific HttpStream subclass.  It holds a
 // non-owning pointer to a QuicChromiumClientStream which it uses to
 // send and receive data.
 class NET_EXPORT_PRIVATE QuicHttpStream
     : public QuicChromiumClientSession::Observer,
       public QuicChromiumClientStream::Delegate,
       public QuicClientPushPromiseIndex::Delegate,
       public MultiplexedHttpStream {
  public:
   QuicHttpStream(const base::WeakPtr<QuicChromiumClientSession>& session,
-                 HttpServerProperties* http_server_properties);
+                 HttpServerProperties* http_server_properties,
+                 bool mark_quic_broken_when_network_suspected);
 
   ~QuicHttpStream() override;
 
   // HttpStream implementation.
   int InitializeStream(const HttpRequestInfo* request_info,
                        RequestPriority priority,
                        const NetLogWithSource& net_log,
                        const CompletionCallback& callback) override;
   int SendRequest(const HttpRequestHeaders& request_headers,
                   HttpResponseInfo* response,
@@ skipping 87 matching lines @@
   // Sets |response_status_| to |response_status| and sets
   // |has_response_status_| to true.
   void SetResponseStatus(int response_status);
   // Computes the correct response status based on the status of the handshake,
   // |session_error|, |connection_error| and |stream_error|.
   int ComputeResponseStatus() const;
 
   State next_state_;
 
   base::WeakPtr<QuicChromiumClientSession> session_;
+  QuicServerId server_id_;  // The ID of the QUIC server for this stream.

[Bence, 2017/04/03 19:02:59]

This member should be const.

[Ryan Hamilton, 2017/04/05 19:26:20]

Done.

 
-  HttpServerProperties* http_server_properties_;
+  HttpServerProperties* http_server_properties_;  // Unowned.
+  // True if QUIC should be marked as broken when a stream is closed with
+  // a possibly network-caused error.
+  bool mark_quic_broken_when_network_suspected_;
 
   QuicVersion quic_version_;
   int session_error_;             // Error code from the connection shutdown.
   bool was_handshake_confirmed_;  // True if the crypto handshake succeeded.
   QuicChromiumClientSession::StreamRequest stream_request_;
   QuicChromiumClientStream* stream_;  // Non-owning.
 
   // The following three fields are all owned by the caller and must
   // outlive this object, according to the HttpStream contract.
 
@@ skipping 70 matching lines @@
   LoadTimingInfo::ConnectTiming connect_timing_;
 
   base::WeakPtrFactory<QuicHttpStream> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(QuicHttpStream);
 };
 
 }  // namespace net
 
 #endif  // NET_QUIC_CHROMIUM_QUIC_HTTP_STREAM_H_