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

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

Issue 2754003002: [DO NOT SUBMIT, PATCH on commit 7fa349e632a44c152b05ca6a66ade5f2e5b3f139] (Closed)
Patch Set: add status logging in SESSION, check clock skewed Created 3 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
« no previous file with comments | « net/quic/chromium/quic_chromium_client_session.h ('k') | net/quic/chromium/quic_stream_factory.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/chromium/quic_chromium_client_session.cc
diff --git a/net/quic/chromium/quic_chromium_client_session.cc b/net/quic/chromium/quic_chromium_client_session.cc
index 9d586835a06a7642bc602cc248e29754485ceb92..8cdbf2b5703f49ea27d6575380c67f3434fc2d26 100644
--- a/net/quic/chromium/quic_chromium_client_session.cc
+++ b/net/quic/chromium/quic_chromium_client_session.cc
@@ -250,6 +250,12 @@ QuicChromiumClientSession::QuicChromiumClientSession(
std::unique_ptr<SocketPerformanceWatcher> socket_performance_watcher,
NetLog* net_log)
: QuicClientSessionBase(connection, push_promise_index, config),
+ received_go_away_(false),
+ received_read_error_(false),
+ connection_closed_(false),
+ notified_session_going_away_(false),
+ notified_session_closed_later_(false),
+ notified_session_closed_(false),
server_id_(server_id),
require_confirmation_(require_confirmation),
stream_factory_(stream_factory),
@@ -926,6 +932,7 @@ void QuicChromiumClientSession::OnCryptoHandshakeMessageReceived(
}
void QuicChromiumClientSession::OnGoAway(const QuicGoAwayFrame& frame) {
+ received_go_away_ = true;
QuicSession::OnGoAway(frame);
NotifyFactoryOfSessionGoingAway();
port_migration_detected_ = frame.error_code == QUIC_ERROR_MIGRATING_PORT;
@@ -940,6 +947,7 @@ void QuicChromiumClientSession::OnConnectionClosed(
QuicErrorCode error,
const std::string& error_details,
ConnectionCloseSource source) {
+ connection_closed_ = true;
DCHECK(!connection()->connected());
logger_->OnConnectionClosed(error, error_details, source);
if (source == ConnectionCloseSource::FROM_PEER) {
@@ -1336,6 +1344,7 @@ QuicChromiumClientSession::GetWeakPtr() {
void QuicChromiumClientSession::OnReadError(
int result,
const DatagramClientSocket* socket) {
+ received_read_error_ = true;
DCHECK(socket != nullptr);
if (socket != GetDefaultSocket()) {
// Ignore read errors from old sockets that are no longer active.
@@ -1363,12 +1372,14 @@ bool QuicChromiumClientSession::OnPacket(const QuicReceivedPacket& packet,
}
void QuicChromiumClientSession::NotifyFactoryOfSessionGoingAway() {
+ notified_session_going_away_ = true;
going_away_ = true;
if (stream_factory_)
stream_factory_->OnSessionGoingAway(this);
}
void QuicChromiumClientSession::NotifyFactoryOfSessionClosedLater() {
+ notified_session_closed_later_ = true;
if (!dynamic_streams().empty())
RecordUnexpectedOpenStreams(NOTIFY_FACTORY_OF_SESSION_CLOSED_LATER);
@@ -1385,6 +1396,7 @@ void QuicChromiumClientSession::NotifyFactoryOfSessionClosedLater() {
}
void QuicChromiumClientSession::NotifyFactoryOfSessionClosed() {
+ notified_session_closed_ = true;
if (!dynamic_streams().empty())
RecordUnexpectedOpenStreams(NOTIFY_FACTORY_OF_SESSION_CLOSED);
« no previous file with comments | « net/quic/chromium/quic_chromium_client_session.h ('k') | net/quic/chromium/quic_stream_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698