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

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

Issue 2301453002: Quic use reduced PING timeout when previous connection times out with open streams (Closed)
Patch Set: address rch's comments Created 4 years, 3 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_stream_factory.h
diff --git a/net/quic/chromium/quic_stream_factory.h b/net/quic/chromium/quic_stream_factory.h
index b0fef1fb7c8e3591cbfe692c9bfe437e574b8e62..f27c0a138d189b6c26181f0d1c44c8a36f67f4d8 100644
--- a/net/quic/chromium/quic_stream_factory.h
+++ b/net/quic/chromium/quic_stream_factory.h
@@ -190,6 +190,7 @@ class NET_EXPORT_PRIVATE QuicStreamFactory
bool close_sessions_on_ip_change,
bool disable_quic_on_timeout_with_open_streams,
int idle_connection_timeout_seconds,
+ int reduced_ping_timeout_seconds,
int packet_reader_yield_after_duration_milliseconds,
bool migrate_sessions_on_network_change,
bool migrate_sessions_early,
@@ -254,6 +255,9 @@ class NET_EXPORT_PRIVATE QuicStreamFactory
// Called by a session after it shuts down.
void OnSessionClosed(QuicChromiumClientSession* session);
+ // Called by a session when connection times out with open streams.
Ryan Hamilton 2016/09/01 19:52:15 nit: s/connection/it/
Zhongyi Shi 2016/09/02 00:48:21 Done.
+ void OnTimeoutWithOpenStreams();
+
// Cancels a pending request.
void CancelRequest(QuicStreamRequest* request);
@@ -596,6 +600,10 @@ class NET_EXPORT_PRIVATE QuicStreamFactory
// 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_;
+
// If more than |yield_after_packets_| packets have been read or more than
// |yield_after_duration_| time has passed, then
// QuicChromiumPacketReader::StartReading() yields by doing a PostTask().

Powered by Google App Engine
This is Rietveld 408576698