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

Unified Diff: net/quic/quic_chromium_client_session.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: add test in quic_stream_factory_test: go away with migration with port only 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_chromium_client_session.h
diff --git a/net/quic/quic_chromium_client_session.h b/net/quic/quic_chromium_client_session.h
index c4ef29387fc18b2283d36946893e09ed2e3dda49..f76b704bc24c2c05a9386b4f54b0bb628e194b43 100644
--- a/net/quic/quic_chromium_client_session.h
+++ b/net/quic/quic_chromium_client_session.h
@@ -20,6 +20,7 @@
#include "base/memory/scoped_ptr.h"
#include "base/time/time.h"
#include "net/base/completion_callback.h"
+#include "net/base/net_error_details.h"
#include "net/base/socket_performance_watcher.h"
#include "net/cert/ct_verify_result.h"
#include "net/proxy/proxy_server.h"
@@ -92,6 +93,9 @@ class NET_EXPORT_PRIVATE QuicChromiumClientSession
// repeatedly.
void CancelRequest();
+ // Populates network error details for this stream request.
+ void PopulateNetErrorDetails(NetErrorDetails* details);
+
private:
friend class QuicChromiumClientSession;
@@ -248,6 +252,9 @@ class NET_EXPORT_PRIVATE QuicChromiumClientSession
scoped_ptr<QuicChromiumPacketReader> reader,
scoped_ptr<QuicPacketWriter> writer);
+ // Populates network error details for this session.
+ void PopulateNetErrorDetails(NetErrorDetails* details);
+
// Returns current default socket. This is the socket over which all
// QUIC packets are sent. This default socket can change, so do not store the
// returned socket.
@@ -328,6 +335,8 @@ class NET_EXPORT_PRIVATE QuicChromiumClientSession
// True when the session is going away, and streams may no longer be created
// on this session. Existing stream will continue to be processed.
bool going_away_;
+ // True when the session receives a go away from server due to port migration.
+ bool received_goaway_because_of_migrating_port_;
Ryan Hamilton 2016/03/10 22:46:44 nit: port_migration_failed?
Zhongyi Shi 2016/03/11 03:56:32 Done.
QuicDisabledReason disabled_reason_;
TokenBindingSignatureMap token_binding_signatures_;
base::WeakPtrFactory<QuicChromiumClientSession> weak_factory_;

Powered by Google App Engine
This is Rietveld 408576698