Minor cleanups in JingleSession and JingleSessionManager.

remoting/protocol/connection_to_client.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/protocol/connection_to_client.h"
 
 #include "google/protobuf/message.h"
 #include "net/base/io_buffer.h"
 #include "remoting/protocol/client_control_sender.h"
 #include "remoting/protocol/host_message_dispatcher.h"
@@ skipping 40 matching lines @@
   // This method can be called from main thread so perform threading switching.
   if (MessageLoop::current() != loop_) {
     loop_->PostTask(
         FROM_HERE,
         NewRunnableMethod(this, &ConnectionToClient::Disconnect));
     return;
   }
 
   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, causing it to close"?

[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) {
   handler_->OnSequenceNumberUpdated(this, sequence_number);
 }
 
 VideoStub* ConnectionToClient::video_stub() {
   return video_writer_.get();
 }
 
@@ skipping 45 matching lines @@
 
 void ConnectionToClient::CloseChannels() {
   if (video_writer_.get())
     video_writer_->Close();
   if (client_control_sender_.get())
     client_control_sender_->Close();
 }
 
 }  // namespace protocol
 }  // namespace remoting