[Chromoting] Add a simple Windows app that registers and starts a host.

remoting/host/setup/win/host_configurer_window.h

+// 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.
+
+#ifndef REMOTING_HOST_SETUP_WIN_HOST_CONFIGURER_WINDOW_H_
+#define REMOTING_HOST_SETUP_WIN_HOST_CONFIGURER_WINDOW_H_
+
+// altbase.h must be included before atlapp.h
+#include <atlbase.h>
+#include <atlapp.h>
+#include <atlcrack.h>
+#include <atluser.h>
+#include <atlwin.h>
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/callback.h"
+#include "net/url_request/url_request_context_getter.h"
+#include "remoting/host/setup/win/host_configurer_resource.h"
+
+namespace remoting {
+
+class HostConfigurerWindow : public ATL::CDialogImpl<HostConfigurerWindow> {

[alexeypa (please no reviews), 2012/10/15 18:48:55]

nit: Add a comment explaining that this class is a modeless dialog, otherwise it
is not obvious why you use DestroyWindow().

[simonmorris, 2012/10/15 21:20:14]

Done.

+ public:
+  enum { IDD = IDD_MAIN };
+
+  BEGIN_MSG_MAP_EX(HostConfigurerWindow)
+    MSG_WM_INITDIALOG(OnInitDialog)
+    MSG_WM_CLOSE(OnClose)
+    COMMAND_ID_HANDLER_EX(IDC_START_HOST, OnStartHost)
+  END_MSG_MAP()
+
+  HostConfigurerWindow(

[alexeypa (please no reviews), 2012/10/15 18:48:55]

nit: I suspect all methods of this class should be called on the UI thread. It
would be nice to document it and add DCHECKs to validate it in runtime.

[simonmorris, 2012/10/15 21:20:14]

Done.

+      scoped_refptr<net::URLRequestContextGetter> url_request_context_getter,
+      scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner,
+      base::Closure quit);
+
+ private:
+  void OnCancel(UINT code, int id, HWND control);
+  void OnClose();
+  LRESULT OnInitDialog(HWND wparam, LPARAM lparam);
+  void OnOk(UINT code, int id, HWND control);
+  void OnStartHost(UINT code, int id, HWND control);
+
+  // Destroys the window and quits the UI message loop.
+  void Finish();
+
+  // Sets an appropriate initial position for the dialog.
+  void PositionWindow();
+
+  // Context needed to start the host.
+  scoped_refptr<net::URLRequestContextGetter> url_request_context_getter_;
+
+  // The UI task runner.
+  scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner_;
+
+  // A task that quits the UI message loop.
+  base::Closure quit_;

[alexeypa (please no reviews), 2012/10/15 18:48:55]

nit: You don't have to mention that |quit_| stop the message loop. You can just
say that it notifies the caller that the window has been destroyed. It will be
up to the caller to decide what to do.

[simonmorris, 2012/10/15 21:20:14]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(HostConfigurerWindow);
+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_HOST_SETUP_WIN_HOST_CONFIGURER_WINDOW_H_