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

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

Issue 2460163002: Refactor QuicServerSessionBase::Visitor (Closed)
Patch Set: Updated patchset dependency Created 4 years, 2 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_simple_server_stream_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 1d2ba1a8d301f71ce9eb179534cfd41e7bf0ef8f..e036564bfab75453be72e338c1f1f079a7465e90 100644
--- a/net/tools/quic/quic_time_wait_list_manager.h
+++ b/net/tools/quic/quic_time_wait_list_manager.h
@@ -21,7 +21,7 @@
#include "net/quic/core/quic_framer.h"
#include "net/quic/core/quic_packet_writer.h"
#include "net/quic/core/quic_protocol.h"
-#include "net/quic/core/quic_server_session_base.h"
+#include "net/quic/core/quic_session.h"
namespace net {
@@ -41,12 +41,19 @@ class QuicTimeWaitListManagerPeer;
// connection_id.
class QuicTimeWaitListManager : public QuicBlockedWriterInterface {
public:
+ class Visitor : public QuicSession::Visitor {
+ public:
+ // Called after the given connection is added to the time-wait std::list.
+ virtual void OnConnectionAddedToTimeWaitList(
+ QuicConnectionId connection_id) = 0;
+ };
+
// writer - the entity that writes to the socket. (Owned by the dispatcher)
// visitor - the entity that manages blocked writers. (The dispatcher)
// helper - provides a clock (Owned by the dispatcher)
// alarm_factory - used to run clean up alarms. (Owned by the dispatcher)
QuicTimeWaitListManager(QuicPacketWriter* writer,
- QuicServerSessionBase::Visitor* visitor,
+ Visitor* visitor,
QuicConnectionHelperInterface* helper,
QuicAlarmFactory* alarm_factory);
~QuicTimeWaitListManager() override;
@@ -196,7 +203,7 @@ class QuicTimeWaitListManager : public QuicBlockedWriterInterface {
QuicPacketWriter* writer_;
// Interface that manages blocked writers.
- QuicServerSessionBase::Visitor* visitor_;
+ Visitor* visitor_;
DISALLOW_COPY_AND_ASSIGN(QuicTimeWaitListManager);
};
« no previous file with comments | « net/tools/quic/quic_simple_server_stream_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