Give DisconnectWindow some TLC.

remoting/host/disconnect_window.h

 // Copyright (c) 2012 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.
 
 #ifndef REMOTING_HOST_DISCONNECT_WINDOW_H_
 #define REMOTING_HOST_DISCONNECT_WINDOW_H_
 
 #include <string>
 
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
 
 namespace remoting {
 
 struct UiStrings;
 
 class DisconnectWindow {
  public:
   enum {
     kMaximumConnectedNameWidthInPixels = 400
   };
 
   // DisconnectCallback is called when the user clicks on the Disconnect button
-  // to disconnect the session. This callback is provided as a parameter to the
-  // Show() method, and will be triggered on the UI thread.
+  // or presses the Disconnect hot-key sequence.

[Jamie, 2012/12/19 00:11:50]

Clarify that it will also be called if the user manages to hide the window via
some other nefarious means. Or, since that's covered below, make this comment
more general, or omit it entirely; perhaps even get rid of the typedef and use
base::Closure.

[Wez, 2012/12/19 00:26:06]

Done.

   typedef base::Callback<void(void)> DisconnectCallback;
 
   virtual ~DisconnectWindow() {}
 
-  // Shows the disconnect window allowing the user to disconnect the session.
-  // Returns false if the window could not be shown for any reason. The
-  // disconnect callback will be invoked if the user clicks the disconnect
-  // button, or if the window is closed for any reason other than Hide() being
-  // called
+  // Shows the disconnect window, allowing the user to disconnect the session.
+  // The window will display text from |ui_strings| and |username|.
+  // |disconnect_callback| will be invoked on the calling UI thread when the
+  // user chooses to disconnect, or if the window is closed by any means other
+  // than Hide(), or deletion of the DisconnectWindow instance.
+  // Show returns false if the window cannot be shown, in which case the
+  // callback will not be invoked.
   virtual bool Show(const UiStrings& ui_strings,
                     const DisconnectCallback& disconnect_callback,
                     const std::string& username) = 0;
 
   // Hides the disconnect window. The disconnect callback will not be invoked.
   virtual void Hide() = 0;
 
   static scoped_ptr<DisconnectWindow> Create();
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_DISCONNECT_WINDOW_H_