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

Unified Diff: net/quic/core/quic_connection.h

Issue 2512793002: Move QuicVersionNegotiationState into QuicConnection, the only place it is used. No behavior change. (Closed)
Patch Set: Created 4 years, 1 month 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 | net/quic/core/quic_protocol.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/core/quic_connection.h
diff --git a/net/quic/core/quic_connection.h b/net/quic/core/quic_connection.h
index 456eec5fe609e4af22a2e99abe7dc1fd17512b57..bb86c425b542183c4c52db7d2afb5dd61f16fd1f 100644
--- a/net/quic/core/quic_connection.h
+++ b/net/quic/core/quic_connection.h
@@ -1042,6 +1042,19 @@ class NET_EXPORT_PRIVATE QuicConnection
std::unique_ptr<QuicSentPacketManagerInterface> sent_packet_manager_;
// The state of connection in version negotiation finite state machine.
+ enum QuicVersionNegotiationState {
+ START_NEGOTIATION = 0,
+ // Server-side this implies we've sent a version negotiation packet and are
+ // waiting on the client to select a compatible version. Client-side this
+ // implies we've gotten a version negotiation packet, are retransmitting the
+ // initial packets with a supported version and are waiting for our first
+ // packet from the server.
+ NEGOTIATION_IN_PROGRESS,
+ // This indicates this endpoint has received a packet from the peer with a
+ // version this endpoint supports. Version negotiation is complete, and the
+ // version number will no longer be sent with future packets.
+ NEGOTIATED_VERSION
+ };
QuicVersionNegotiationState version_negotiation_state_;
// Tracks if the connection was created by the server or the client.
« no previous file with comments | « no previous file | net/quic/core/quic_protocol.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698