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

This gets rid of the DesktopEnvironment::OnPause() notification from the Host.
Instead, that code is moved to DesktopEnvironment::ContinueSession() and
provided as a callback function parameter to ContinueWindow::Show().

This will simplify a future CL to move the ContinueWindow/DisconnectWindow code
into an IT2Me-specific HostObserver subclass.


BUG=None
TEST=No change in functionality.


Review URL: http://codereview.chromium.org/8624009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111443 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 10 insertions, 1 deletions

diff --git a/remoting/host/continue_window.h b/remoting/host/continue_window.h
index 0c52416..1840664 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,
+                    const ContinueSessionCallback& callback) = 0;
 
   // Hide the continuation window if it is visible.
   virtual void Hide() = 0;