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

Unified Diff: net/tools/quic/quic_time_wait_list_manager.h

Issue 1138443003: Land Recent QUIC Changes until 05/13/2015 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: compile error fixes Created 5 years, 7 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
« no previous file with comments | « net/tools/quic/quic_server_session_test.cc ('k') | net/tools/quic/quic_time_wait_list_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/quic/quic_time_wait_list_manager.h
diff --git a/net/tools/quic/quic_time_wait_list_manager.h b/net/tools/quic/quic_time_wait_list_manager.h
index 8c128698c97bca35635b4cde5217377c6b197af1..bbaf6f01562bd3631e3c9dde7eeb6a1f16e76627 100644
--- a/net/tools/quic/quic_time_wait_list_manager.h
+++ b/net/tools/quic/quic_time_wait_list_manager.h
@@ -55,9 +55,16 @@ class QuicTimeWaitListManager : public QuicBlockedWriterInterface {
// and sends it again when packets are received for added connection_ids. If
// nullptr, a public reset packet is sent with the specified |version|.
// DCHECKs that connection_id is not already on the list. "virtual" to
- // override in tests.
+ // override in tests. If "connection_rejected_statelessly" is true, it means
+ // that the connection was closed due to a stateless reject, and no close
+ // packet is expected. Any packets that are received for connection_id will
+ // be black-holed.
+ // TODO(jokulik): In the future, we plan send (redundant) SREJ packets back to
+ // the client in response to stray data-packets that arrive after the first
+ // SREJ. This requires some new plumbing, so we black-hole for now.
virtual void AddConnectionIdToTimeWait(QuicConnectionId connection_id,
QuicVersion version,
+ bool connection_rejected_statelessly,
QuicEncryptedPacket* close_packet);
// Returns true if the connection_id is in time wait state, false otherwise.
@@ -145,15 +152,18 @@ class QuicTimeWaitListManager : public QuicBlockedWriterInterface {
ConnectionIdData(int num_packets_,
QuicVersion version_,
QuicTime time_added_,
- QuicEncryptedPacket* close_packet)
+ QuicEncryptedPacket* close_packet,
+ bool connection_rejected_statelessly)
: num_packets(num_packets_),
version(version_),
time_added(time_added_),
- close_packet(close_packet) {}
+ close_packet(close_packet),
+ connection_rejected_statelessly(connection_rejected_statelessly) {}
int num_packets;
QuicVersion version;
QuicTime time_added;
QuicEncryptedPacket* close_packet;
+ bool connection_rejected_statelessly;
};
// linked_hash_map allows lookup by ConnectionId and traversal in add order.
« no previous file with comments | « net/tools/quic/quic_server_session_test.cc ('k') | net/tools/quic/quic_time_wait_list_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698