Made DesktopEnvironment responsible for creation of the disconnect window.

remoting/host/disconnect_window_win.cc

 // Copyright (c) 2012 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.
 
-#include "remoting/host/disconnect_window.h"
-
 #include <windows.h>
 
+#include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/process_util.h"
+#include "base/single_thread_task_runner.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "base/win/scoped_gdi_object.h"
 #include "base/win/scoped_hdc.h"
 #include "base/win/scoped_select_object.h"
+#include "remoting/host/host_window.h"
 #include "remoting/host/ui_strings.h"
 #include "remoting/host/win/core_resource.h"
 
-// TODO(garykac): Lots of duplicated code in this file and
-// continue_window_win.cc. If we need to expand this then we should
-// create a class with the shared code.
+namespace remoting {
 
 namespace {
 
 const int DISCONNECT_HOTKEY_ID = 1000;
 
 // Maximum length of "Your desktop is shared with ..." message in UTF-16
 // characters.
 const size_t kMaxSharingWithTextLength = 100;
 
 const wchar_t kShellTrayWindowName[] = L"Shell_TrayWnd";
 const int kWindowBorderRadius = 14;
 
-} // namespace anonymous
+class DisconnectWindowWin : public HostWindow::Core {
+ public:
+  DisconnectWindowWin(
+      scoped_refptr<base::SingleThreadTaskRunner> caller_task_runner,
+      scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner,
+      const base::Closure& disconnect_callback,
+      const std::string& username,
+      const UiStrings& ui_strings);
 
-namespace remoting {
-
-class DisconnectWindowWin : public DisconnectWindow {
- public:
-  explicit DisconnectWindowWin(const UiStrings* ui_strings);
+ protected:
+  friend class base::RefCountedThreadSafe<Core>;
   virtual ~DisconnectWindowWin();
 
-  // DisconnectWindow interface.
-  virtual bool Show(const base::Closure& disconnect_callback,
-                    const std::string& username) OVERRIDE;
-  virtual void Hide() OVERRIDE;
+  // HostWindow::Core overrides.
+  virtual void StartOnUiThread() OVERRIDE;
+  virtual void StopOnUiThread() OVERRIDE;
 
-private:
   static INT_PTR CALLBACK DialogProc(HWND hwnd, UINT message, WPARAM wparam,
                                      LPARAM lparam);
 
   BOOL OnDialogMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam);
 
   // Creates the dialog window and registers the disconnect hot key.
-  bool BeginDialog(const std::string& username);
+  bool BeginDialog();
 
   // Closes the dialog, unregisters the hot key and invokes the disconnect
   // callback, if set.
   void EndDialog();
 
   // Trys to position the dialog window above the taskbar.
   void SetDialogPosition();
 
   // Applies localization string and resizes the dialog.
-  bool SetStrings(const string16& username);
+  bool SetStrings();
 
+  // Invoked in the |caller_task_runner_| thread to disconnect the client
+  // session.
   base::Closure disconnect_callback_;
+
+  // Specifies the remote user name.
+  std::string username_;
+
   HWND hwnd_;
   bool has_hotkey_;
   base::win::ScopedGDIObject<HPEN> border_pen_;
 
-  // Points to the localized strings.
-  const UiStrings* ui_strings_;
-
   DISALLOW_COPY_AND_ASSIGN(DisconnectWindowWin);
 };
 
-static int GetControlTextWidth(HWND control) {
+int GetControlTextWidth(HWND control) {
   RECT rect = {0, 0, 0, 0};
   WCHAR text[256];
   int result = GetWindowText(control, text, arraysize(text));
   if (result) {
     base::win::ScopedGetDC dc(control);
     base::win::ScopedSelectObject font(
         dc, (HFONT)SendMessage(control, WM_GETFONT, 0, 0));
     DrawText(dc, text, -1, &rect, DT_CALCRECT | DT_SINGLELINE);
   }
   return rect.right;
 }
 
-DisconnectWindowWin::DisconnectWindowWin(const UiStrings* ui_strings)
-    : hwnd_(NULL),
+DisconnectWindowWin::DisconnectWindowWin(
+    scoped_refptr<base::SingleThreadTaskRunner> caller_task_runner,
+    scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner,
+    const base::Closure& disconnect_callback,
+    const std::string& username,
+    const UiStrings& ui_strings)
+    : HostWindow::Core(caller_task_runner,
+                       ui_task_runner,
+                       ui_strings),
+      disconnect_callback_(disconnect_callback),
+      username_(username),
+      hwnd_(NULL),
       has_hotkey_(false),
       border_pen_(CreatePen(PS_SOLID, 5,
-                            RGB(0.13 * 255, 0.69 * 255, 0.11 * 255))),
-      ui_strings_(ui_strings) {
+                            RGB(0.13 * 255, 0.69 * 255, 0.11 * 255))) {
 }
 
 DisconnectWindowWin::~DisconnectWindowWin() {
-  Hide();
 }
 
-bool DisconnectWindowWin::Show(const base::Closure& disconnect_callback,
-                               const std::string& username) {
-  DCHECK(disconnect_callback_.is_null());
-  DCHECK(!disconnect_callback.is_null());
+void DisconnectWindowWin::StartOnUiThread() {
+  DCHECK(ui_task_runner()->BelongsToCurrentThread());
 
-  disconnect_callback_ = disconnect_callback;
-
-  if (BeginDialog(username)) {
-    return true;
-  } else {
-    Hide();
-    return false;
+  if (!BeginDialog()) {
+    StopOnUiThread();
+    caller_task_runner()->PostTask(FROM_HERE, disconnect_callback_);
   }
 }
 
-void DisconnectWindowWin::Hide() {
-  // Clear the |disconnect_callback_| so it won't be invoked by EndDialog().
-  disconnect_callback_.Reset();
+void DisconnectWindowWin::StopOnUiThread() {
+  DCHECK(ui_task_runner()->BelongsToCurrentThread());
+
   EndDialog();
 }
 
-INT_PTR CALLBACK DisconnectWindowWin::DialogProc(HWND hwnd, UINT message,
-                                                 WPARAM wparam, LPARAM lparam) {
+INT_PTR CALLBACK DisconnectWindowWin::DialogProc(HWND hwnd,
+                                                 UINT message,
+                                                 WPARAM wparam,
+                                                 LPARAM lparam) {
   LONG_PTR self = NULL;
   if (message == WM_INITDIALOG) {
     self = lparam;
 
     // Store |this| to the window's user data.
     SetLastError(ERROR_SUCCESS);
     LONG_PTR result = SetWindowLongPtr(hwnd, DWLP_USER, self);
     if (result == 0 && GetLastError() != ERROR_SUCCESS)
       reinterpret_cast<DisconnectWindowWin*>(self)->EndDialog();
   } else {
     self = GetWindowLongPtr(hwnd, DWLP_USER);
   }
 
   if (self) {
     return reinterpret_cast<DisconnectWindowWin*>(self)->OnDialogMessage(
         hwnd, message, wparam, lparam);
   }
   return FALSE;
 }
 
-BOOL DisconnectWindowWin::OnDialogMessage(HWND hwnd, UINT message,
-                                          WPARAM wparam, LPARAM lparam) {
+BOOL DisconnectWindowWin::OnDialogMessage(HWND hwnd,
+                                          UINT message,
+                                          WPARAM wparam,
+                                          LPARAM lparam) {
+  DCHECK(ui_task_runner()->BelongsToCurrentThread());
+
   switch (message) {
     // Ignore close messages.
     case WM_CLOSE:
       return TRUE;
 
     // Handle the Disconnect button.
     case WM_COMMAND:
       switch (LOWORD(wparam)) {
         case IDC_DISCONNECT:
           EndDialog();
@@ skipping 44 matching lines @@
         RoundRect(hdc, rect.left, rect.top, rect.right - 1, rect.bottom - 1,
                   kWindowBorderRadius, kWindowBorderRadius);
       }
       EndPaint(hwnd_, &ps);
       return TRUE;
     }
   }
   return FALSE;
 }
 
-bool DisconnectWindowWin::BeginDialog(const std::string& username) {
+bool DisconnectWindowWin::BeginDialog() {
+  DCHECK(ui_task_runner()->BelongsToCurrentThread());
   DCHECK(!hwnd_);
 
   // Load the dialog resource so that we can modify the RTL flags if necessary.
   HMODULE module = base::GetModuleFromAddress(&DialogProc);
   HRSRC dialog_resource =
       FindResource(module, MAKEINTRESOURCE(IDD_DISCONNECT), RT_DIALOG);
   if (!dialog_resource)
     return false;
 
   HGLOBAL dialog_template = LoadResource(module, dialog_resource);
   if (!dialog_template)
     return false;
 
   DLGTEMPLATE* dialog_pointer =
       reinterpret_cast<DLGTEMPLATE*>(LockResource(dialog_template));
   if (!dialog_pointer)
     return false;
 
   // The actual resource type is DLGTEMPLATEEX, but this is not defined in any
   // standard headers, so we treat it as a generic pointer and manipulate the
   // correct offsets explicitly.
   scoped_array<unsigned char> rtl_dialog_template;
-  if (ui_strings_->direction == UiStrings::RTL) {
+  if (ui_strings().direction == UiStrings::RTL) {
     unsigned long dialog_template_size =
         SizeofResource(module, dialog_resource);
     rtl_dialog_template.reset(new unsigned char[dialog_template_size]);
     memcpy(rtl_dialog_template.get(), dialog_pointer, dialog_template_size);
     DWORD* rtl_dwords = reinterpret_cast<DWORD*>(rtl_dialog_template.get());
     rtl_dwords[2] |= (WS_EX_LAYOUTRTL | WS_EX_RTLREADING);
     dialog_pointer = reinterpret_cast<DLGTEMPLATE*>(rtl_dwords);
   }
 
   hwnd_ = CreateDialogIndirectParam(module, dialog_pointer, NULL,
                                     DialogProc, reinterpret_cast<LPARAM>(this));
   if (!hwnd_)
     return false;
 
   // Set up handler for Ctrl-Alt-Esc shortcut.
   if (!has_hotkey_ && RegisterHotKey(hwnd_, DISCONNECT_HOTKEY_ID,
                                      MOD_ALT | MOD_CONTROL, VK_ESCAPE)) {
     has_hotkey_ = true;
   }
 
-  if (!SetStrings(UTF8ToUTF16(username)))
+  if (!SetStrings())
     return false;
 
   SetDialogPosition();
   ShowWindow(hwnd_, SW_SHOW);
   return IsWindowVisible(hwnd_) != FALSE;
 }
 
 void DisconnectWindowWin::EndDialog() {
+  DCHECK(ui_task_runner()->BelongsToCurrentThread());
+
   if (has_hotkey_) {
     UnregisterHotKey(hwnd_, DISCONNECT_HOTKEY_ID);
     has_hotkey_ = false;
   }
 
   if (hwnd_) {
     DestroyWindow(hwnd_);
     hwnd_ = NULL;
   }
 
-  if (!disconnect_callback_.is_null()) {
-    disconnect_callback_.Run();
-    disconnect_callback_.Reset();
-  }
+  caller_task_runner()->PostTask(FROM_HERE, disconnect_callback_);
 }
 
 void DisconnectWindowWin::SetDialogPosition() {
+  DCHECK(ui_task_runner()->BelongsToCurrentThread());
+
   // Try to center the window above the task-bar. If that fails, use the
   // primary monitor. If that fails (very unlikely), use the default position.
   HWND taskbar = FindWindow(kShellTrayWindowName, NULL);
   HMONITOR monitor = MonitorFromWindow(taskbar, MONITOR_DEFAULTTOPRIMARY);
   MONITORINFO monitor_info = {sizeof(monitor_info)};
   RECT window_rect;
   if (GetMonitorInfo(monitor, &monitor_info) &&
       GetWindowRect(hwnd_, &window_rect)) {
     int window_width = window_rect.right - window_rect.left;
     int window_height = window_rect.bottom - window_rect.top;
     int top = monitor_info.rcWork.bottom - window_height;
     int left = (monitor_info.rcWork.right + monitor_info.rcWork.left -
         window_width) / 2;
     SetWindowPos(hwnd_, NULL, left, top, 0, 0, SWP_NOSIZE | SWP_NOZORDER);
   }
 }
 
-bool DisconnectWindowWin::SetStrings(const string16& username) {
-  if (!SetWindowText(hwnd_, ui_strings_->product_name.c_str()))
+bool DisconnectWindowWin::SetStrings() {
+  DCHECK(ui_task_runner()->BelongsToCurrentThread());
+
+  if (!SetWindowText(hwnd_, ui_strings().product_name.c_str()))
     return false;
 
   // Localize the disconnect button text and measure length of the old and new
   // labels.
   HWND disconnect_button = GetDlgItem(hwnd_, IDC_DISCONNECT);
   if (!disconnect_button)
     return false;
   int button_old_required_width = GetControlTextWidth(disconnect_button);
   if (!SetWindowText(disconnect_button,
-                     ui_strings_->disconnect_button_text.c_str())) {
+                     ui_strings().disconnect_button_text.c_str())) {
     return false;
   }
   int button_new_required_width = GetControlTextWidth(disconnect_button);
   int button_width_delta =
       button_new_required_width - button_old_required_width;
 
   // Format and truncate "Your desktop is shared with ..." message.
-  string16 text = ReplaceStringPlaceholders(ui_strings_->disconnect_message,
-                                            username, NULL);
+  string16 text = ReplaceStringPlaceholders(ui_strings().disconnect_message,
+                                            UTF8ToUTF16(username_), NULL);
   if (text.length() > kMaxSharingWithTextLength)
     text.erase(kMaxSharingWithTextLength);
 
   // Set localized and truncated "Your desktop is shared with ..." message and
   // measure length of the old and new text.
   HWND sharing_with_label = GetDlgItem(hwnd_, IDC_DISCONNECT_SHARINGWITH);
   if (!sharing_with_label)
     return false;
   int label_old_required_width = GetControlTextWidth(sharing_with_label);
   if (!SetWindowText(sharing_with_label, text.c_str()))
@@ skipping 47 matching lines @@
   HRGN rgn = CreateRoundRectRgn(0, 0, width, height, kWindowBorderRadius,
                                 kWindowBorderRadius);
   if (!rgn)
     return false;
   if (!SetWindowRgn(hwnd_, rgn, TRUE))
     return false;
 
   return true;
 }
 
-scoped_ptr<DisconnectWindow> DisconnectWindow::Create(
-    const UiStrings* ui_strings) {
-  return scoped_ptr<DisconnectWindow>(new DisconnectWindowWin(ui_strings));
+} // namespace
+
+scoped_ptr<HostWindow> HostWindow::CreateDisconnectWindow(
+    scoped_refptr<base::SingleThreadTaskRunner> caller_task_runner,
+    scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner,
+    const base::Closure& disconnect_callback,
+    const std::string& username,
+    const UiStrings& ui_strings) {
+  scoped_refptr<Core> core = new DisconnectWindowWin(caller_task_runner,
+                                                     ui_task_runner,
+                                                     disconnect_callback,
+                                                     username,
+                                                     ui_strings);
+  return scoped_ptr<HostWindow>(new HostWindow(core));
 }
 
 }  // namespace remoting