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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: remoting/protocol/connection_to_client.cc

Issue 7312013: Minor cleanups in JingleSession and JingleSessionManager. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: - Created 9 years, 5 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 | « remoting/jingle_glue/xmpp_signal_strategy.cc ('k') | remoting/protocol/connection_to_host.cc » ('j') | remoting/protocol/jingle_session.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/connection_to_client.cc
diff --git a/remoting/protocol/connection_to_client.cc b/remoting/protocol/connection_to_client.cc
index 4810d27e2dd7c5f8dc0f3adc229a435d1a05425a..e407c6890878e03eb898ae719b8c0ae1cece0cc9 100644
--- a/remoting/protocol/connection_to_client.cc
+++ b/remoting/protocol/connection_to_client.cc
@@ -59,10 +59,8 @@ void ConnectionToClient::Disconnect() {
CloseChannels();
// If there is a channel then close it and release the reference.
Wez 2011/07/06 21:24:25 nit: Update this comment, e.g. "... release it, ca
Sergey Ulanov 2011/07/06 23:15:01 Done.
- if (session_.get()) {
- session_->Close();
+ if (session_.get())
session_.reset();
- }
}
void ConnectionToClient::UpdateSequenceNumber(int64 sequence_number) {
« no previous file with comments | « remoting/jingle_glue/xmpp_signal_strategy.cc ('k') | remoting/protocol/connection_to_host.cc » ('j') | remoting/protocol/jingle_session.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698