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

remoting/host/continue_window.h

 // 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.
 
 #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;
 
   static ContinueWindow* Create();
 };
 
 }
 
 #endif  // REMOTING_HOST_CONTINUE_WINDOW_H