Give DisconnectWindow some TLC.

remoting/host/disconnect_window.h

Index: remoting/host/disconnect_window.h
diff --git a/remoting/host/disconnect_window.h b/remoting/host/disconnect_window.h
index 51269b77fb47fea5c6bf2057d692a548a89df614..b20e36167dd6ec00116e882508805209532fc31a 100644
--- a/remoting/host/disconnect_window.h
+++ b/remoting/host/disconnect_window.h
@@ -21,17 +21,18 @@ class DisconnectWindow {
   };
 
   // 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;