Turned UiStrings into a singleton so that the continue window does not depend on ChromotingHost.

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 98e534706b7c179740851d8ba05f7ac903c22a92..fa1ae2588fbfcf96463843717d7ffadd4e25f9e0 100644
--- a/remoting/host/continue_window.h
+++ b/remoting/host/continue_window.h
@@ -9,7 +9,7 @@
 
 namespace remoting {
 
-class ChromotingHost;
+struct UiStrings;
 
 class ContinueWindow {
  public:
@@ -23,13 +23,12 @@ class ContinueWindow {
 
   // Show the continuation window requesting that the user approve continuing
   // the session.
-  virtual void Show(ChromotingHost* host,
-                    const ContinueSessionCallback& callback) = 0;
+  virtual void Show(const ContinueSessionCallback& callback) = 0;
 
   // Hide the continuation window if it is visible.
   virtual void Hide() = 0;
 
-  static scoped_ptr<ContinueWindow> Create();
+  static scoped_ptr<ContinueWindow> Create(const UiStrings* ui_strings);

[Lambros, 2013/01/16 02:30:47]

nit: Does it make sense to pass by const ref everywhere?

[alexeypa (please no reviews), 2013/01/16 16:55:35]

The reason for using a pointer that it implies that the pointed value can be
used after the method returns stronger than a reference.

 };
 
 }  // namespace remoting