Made DesktopEnvironment responsible for creation of the disconnect window.

remoting/host/basic_desktop_environment.h

Index: remoting/host/basic_desktop_environment.h
diff --git a/remoting/host/basic_desktop_environment.h b/remoting/host/basic_desktop_environment.h
index dcd5f8fc3b5757ee0f57c9ddc4332dd5fd21abd2..f5de55b907b2e2b773bce0108096a39493e545d8 100644
--- a/remoting/host/basic_desktop_environment.h
+++ b/remoting/host/basic_desktop_environment.h
@@ -9,9 +9,11 @@
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "remoting/host/desktop_environment.h"
+#include "remoting/host/ui_strings.h"
 
 namespace remoting {
 
+class HostWindow;
 class LocalInputMonitor;
 
 // Used to create audio/video capturers and event executor that work with
@@ -28,11 +30,15 @@ class BasicDesktopEnvironment : public DesktopEnvironment {
 
  protected:
   friend class BasicDesktopEnvironmentFactory;
+
+  // |ui_strings| are hosted by the BasicDesktopEnvironmentFactory instance that
+  // created |this|. |ui_strings| must outlive this object.
   BasicDesktopEnvironment(
       scoped_refptr<base::SingleThreadTaskRunner> caller_task_runner,
       scoped_refptr<base::SingleThreadTaskRunner> input_task_runner,
       scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner,
-      base::WeakPtr<ClientSessionControl> client_session_control);
+      base::WeakPtr<ClientSessionControl> client_session_control,
+      const UiStrings* ui_strings);

[Sergey Ulanov, 2013/04/02 19:04:22]

Maybe just copy ui_strings, to avoid worrying about object lifetime? it's small.

[alexeypa (please no reviews), 2013/04/03 20:26:43]

Done.

 
   scoped_refptr<base::SingleThreadTaskRunner> caller_task_runner() const {
     return caller_task_runner_;
@@ -46,6 +52,8 @@ class BasicDesktopEnvironment : public DesktopEnvironment {
     return ui_task_runner_;
   }
 
+  const UiStrings* ui_strings() const { return ui_strings_; }

[Sergey Ulanov, 2013/04/02 19:04:22]

nit: return const ref here?

[Sergey Ulanov, 2013/04/02 19:04:22]

is this method used anywhere?

[alexeypa (please no reviews), 2013/04/03 20:26:43]

I removed this code.

+
  private:
   // Task runner on which methods of DesktopEnvironment interface should be
   // called.
@@ -57,9 +65,15 @@ class BasicDesktopEnvironment : public DesktopEnvironment {
   // Used to run UI code.
   scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner_;
 
+  // Presents the disconnect window to the local user.
+  scoped_ptr<HostWindow> disconnect_window_;
+
   // Notifies the client session about the local mouse movements.
   scoped_ptr<LocalInputMonitor> local_input_monitor_;
 
+  // Points to the localized UI strings.
+  const UiStrings* ui_strings_;
+
   DISALLOW_COPY_AND_ASSIGN(BasicDesktopEnvironment);
 };
 
@@ -69,7 +83,8 @@ class BasicDesktopEnvironmentFactory : public DesktopEnvironmentFactory {
   BasicDesktopEnvironmentFactory(
       scoped_refptr<base::SingleThreadTaskRunner> caller_task_runner,
       scoped_refptr<base::SingleThreadTaskRunner> input_task_runner,
-      scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner);
+      scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner,
+      const UiStrings& ui_strings);
   virtual ~BasicDesktopEnvironmentFactory();
 
   // DesktopEnvironmentFactory implementation.
@@ -90,6 +105,8 @@ class BasicDesktopEnvironmentFactory : public DesktopEnvironmentFactory {
     return ui_task_runner_;
   }
 
+  const UiStrings& ui_strings() const { return ui_strings_; }
+
  private:
   // Task runner on which methods of DesktopEnvironmentFactory interface should
   // be called.
@@ -101,6 +118,9 @@ class BasicDesktopEnvironmentFactory : public DesktopEnvironmentFactory {
   // Used to run UI code.
   scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner_;
 
+  // Contains a copy of the localized UI strings.
+  const UiStrings ui_strings_;
+
   DISALLOW_COPY_AND_ASSIGN(BasicDesktopEnvironmentFactory);
 };