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

Unified Diff: net/quic/chromium/quic_http_stream.h

Issue 2789093003: Mark QUIC broken when the network blackholes after the handshake (Closed)
Patch Set: Created 3 years, 9 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/quic/chromium/quic_http_stream.h
diff --git a/net/quic/chromium/quic_http_stream.h b/net/quic/chromium/quic_http_stream.h
index 2ca5bbcfed78744500833c60e124fbd03c2a9bcf..406949fac35aca51aa929742310ba89926a28e27 100644
--- a/net/quic/chromium/quic_http_stream.h
+++ b/net/quic/chromium/quic_http_stream.h
@@ -42,7 +42,8 @@ class NET_EXPORT_PRIVATE QuicHttpStream
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;
@@ -150,8 +151,12 @@ class NET_EXPORT_PRIVATE QuicHttpStream
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.

Powered by Google App Engine
This is Rietveld 408576698