diff options
author | lukasza <lukasza@chromium.org> | 2015-04-30 16:09:59 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-04-30 23:10:33 +0000 |
commit | f34d841fd05a61d8fc5ca1fdf0f4e3f04223f0bf (patch) | |
tree | 1b7d62683b93767cced122950e0658c49669a35a /remoting/host/client_session.cc | |
parent | 2eaeec1d219f20758dbf57f28b00569d632420c4 (diff) | |
download | chromium_src-f34d841fd05a61d8fc5ca1fdf0f4e3f04223f0bf.zip chromium_src-f34d841fd05a61d8fc5ca1fdf0f4e3f04223f0bf.tar.gz chromium_src-f34d841fd05a61d8fc5ca1fdf0f4e3f04223f0bf.tar.bz2 |
Logging client disconnected events (incl. the error code)
to help diagnose stale access token in some app remoting scenarios
(tracked internally at b/20662147)
The new log entry complements an existing one that logs client connected
events in ChromotingHost::OnIncomingSession (chromoting_host.cc:302).
BUG=
Review URL: https://codereview.chromium.org/1117043002
Cr-Commit-Position: refs/heads/master@{#327822}
Diffstat (limited to 'remoting/host/client_session.cc')
-rw-r--r-- | remoting/host/client_session.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/remoting/host/client_session.cc b/remoting/host/client_session.cc index 905539b..10e4d07 100644 --- a/remoting/host/client_session.cc +++ b/remoting/host/client_session.cc @@ -370,6 +370,8 @@ void ClientSession::OnConnectionClosed( DCHECK(CalledOnValidThread()); DCHECK_EQ(connection_.get(), connection); + HOST_LOG << "Client disconnected: " << client_jid_ << "; error = " << error; + // Ignore any further callbacks. weak_factory_.InvalidateWeakPtrs(); @@ -391,7 +393,6 @@ void ClientSession::OnConnectionClosed( desktop_environment_.reset(); // Notify the ChromotingHost that this client is disconnected. - // TODO(sergeyu): Log failure reason? event_handler_->OnSessionClosed(this); } |