Refactor ContinueWindow::Show() to accept a callback parameter.

remoting/host/continue_window.h

Index: remoting/host/continue_window.h
diff --git a/remoting/host/continue_window.h b/remoting/host/continue_window.h
index 0c52416817de502a72bbc2c068f351631adad317..18406643d4429ea88274c7996c10d58e7b1871fa 100644
--- a/remoting/host/continue_window.h
+++ b/remoting/host/continue_window.h
@@ -5,17 +5,26 @@
 #ifndef REMOTING_HOST_CONTINUE_WINDOW_H
 #define REMOTING_HOST_CONTINUE_WINDOW_H
 
+#include "base/callback.h"
+
 namespace remoting {
 
 class ChromotingHost;
 
 class ContinueWindow {
  public:
+  // ContinueSessionCallback is called when the user clicks on the
+  // Continue button to resume the session, or dismisses the window to
+  // terminate the session.  This callback is provided as a parameter to the
+  // Show() method, and will be triggered on the UI thread.
+  typedef base::Callback<void(bool)> ContinueSessionCallback;
+
   virtual ~ContinueWindow() {}
 
   // Show the continuation window requesting that the user approve continuing
   // the session.
-  virtual void Show(ChromotingHost* host) = 0;
+  virtual void Show(ChromotingHost* host,

[Sergey Ulanov, 2011/11/23 19:23:15]

I think the point of this CL is to remove dependency on ChromotingHost from
ContinueWindow. So do we still need this parameter?

[Lambros, 2011/11/23 19:39:28]

My main purpose was to get rid of the unnecessary OnPause() notification, which
was slightly hindering the IT2Me HostObserver refactoring.  Removing this
dependency is a nice side-effect :)  The |host| parameter could be removed and
replaced with a UiStrings parameter.  I'll do this cleanup in a followup CL.

+                    const ContinueSessionCallback& callback) = 0;
 
   // Hide the continuation window if it is visible.
   virtual void Hide() = 0;