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

Unified Diff: net/quic/quic_stream_factory.h

Issue 1775283002: [Domain Reliabiliy: net stack] Plumb received go away from server due to connection migration. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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/quic_stream_factory.h
diff --git a/net/quic/quic_stream_factory.h b/net/quic/quic_stream_factory.h
index 919f9ea784f3a86fe6a7ecb8ee791141866dc51c..cb50aab301f600631403b55768676d164a5300f6 100644
--- a/net/quic/quic_stream_factory.h
+++ b/net/quic/quic_stream_factory.h
@@ -21,6 +21,7 @@
#include "net/base/address_list.h"
#include "net/base/completion_callback.h"
#include "net/base/host_port_pair.h"
+#include "net/base/net_error_details.h"
#include "net/base/network_change_notifier.h"
#include "net/cert/cert_database.h"
#include "net/http/http_server_properties.h"
@@ -206,6 +207,10 @@ class NET_EXPORT_PRIVATE QuicStreamFactory
// created on it.
void OnSessionGoingAway(QuicChromiumClientSession* session);
+ // Called by a session when it is going away due to connection migration.
+ // Log it to net error details.
+ void OnSessionGoingAwayForConnectionMigration();
+
// Called by a session after it shuts down.
void OnSessionClosed(QuicChromiumClientSession* session);
@@ -283,6 +288,9 @@ class NET_EXPORT_PRIVATE QuicStreamFactory
void OnCertAdded(const X509Certificate* cert) override;
void OnCACertChanged(const X509Certificate* cert) override;
+ // Populates network error details for this stream factory.
+ void PopulateNetErrorDetails(NetErrorDetails* details);
+
bool require_confirmation() const { return require_confirmation_; }
void set_require_confirmation(bool require_confirmation);
@@ -456,6 +464,8 @@ class NET_EXPORT_PRIVATE QuicStreamFactory
// config from the disk cache.
bool enable_connection_racing_;
+ bool goaway_received_for_connection_mirgation_;
+
// Set if experimental non-blocking IO should be used on windows sockets.
bool enable_non_blocking_io_;

Powered by Google App Engine
This is Rietveld 408576698