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

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

Issue 2329853002: Introduces ability for session to wait on a migration trigger for a new (Closed)
Patch Set: network notifications handled 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_chromium_client_session.h
diff --git a/net/quic/chromium/quic_chromium_client_session.h b/net/quic/chromium/quic_chromium_client_session.h
index d3eb0fd83f260753c00f7eb2017e6414bcb6a9e3..c06d27d55158371d6c762d50277adcd8b94141bc 100644
--- a/net/quic/chromium/quic_chromium_client_session.h
+++ b/net/quic/chromium/quic_chromium_client_session.h
@@ -273,6 +273,19 @@ class NET_EXPORT_PRIVATE QuicChromiumClientSession
std::unique_ptr<QuicChromiumPacketReader> reader,
std::unique_ptr<QuicChromiumPacketWriter> writer);
+ // Method to call when NetworkChangeNotifier notifies observers of a
+ // new network. Tries to migrate any sessions that are waiting for a
+ // new network to connect.
+ void OnNetworkConnected(NetworkChangeNotifier::NetworkHandle network,
+ const BoundNetLog& bound_net_log);
+
+ // Schedules a migration alarm if one isn't already pending to wait
+ // for a new network.
+ void OnNoNewNetwork();
+
+ // If migration alarm was not cancelled, closes session with error.
+ void OnMigrationTimeout();
+
// Populates network error details for this session.
void PopulateNetErrorDetails(NetErrorDetails* details);
@@ -376,6 +389,7 @@ class NET_EXPORT_PRIVATE QuicChromiumClientSession
scoped_refptr<StringIOBuffer> packet_;
bool migration_pending_; // True while migration is underway.
bool write_pending_; // True while post-migration write is underway.
+ bool migration_alarm_pending_; // True when migration alarm is armed.
base::WeakPtrFactory<QuicChromiumClientSession> weak_factory_;
DISALLOW_COPY_AND_ASSIGN(QuicChromiumClientSession);

Powered by Google App Engine
This is Rietveld 408576698