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

remoting/host/setup/win/host_configurer.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 <windows.h>
+#include <ole2.h>
+
+#include "base/at_exit.h"
+#include "base/command_line.h"
+#include "base/threading/thread.h"
+#include "remoting/host/setup/win/host_configurer_window.h"
+#include "remoting/host/url_request_context.h"
+
+// An app that runs a HostConfigurerWindow.
+int WINAPI WinMain(HINSTANCE instance_handle, HINSTANCE prev_instance_handle,
+                   LPSTR cmd_line, int cmd)
+{
+  // google_apis::GetOAuth2ClientID/Secret need the next line.
+  CommandLine::Init(0, NULL);

[alexeypa (please no reviews), 2012/10/11 16:57:52]

Add a comment explaining why it is acceptable to pass (0, NULL) on Windows.

[simonmorris, 2012/10/11 21:27:35]

Done.

+
+  // This object instance is required by Chrome code (for example,
+  // FilePath, LazyInstance, MessageLoop).
+  base::AtExitManager exit_manager;
+
+  // Provide message loops and threads for the URLRequestContextGetter.

[alexeypa (please no reviews), 2012/10/11 16:57:52]

Take a look at AutoThread and AutoThreadTaskRunner that wez@ have checked in
recently. I believe it is a better way to manage threads.

[simonmorris, 2012/10/11 21:27:35]

I can see they can be useful classes. But I think they're overkill for this
situation. Here, there are only two threads, whose lifetimes are very clear.

+  MessageLoop message_loop(MessageLoop::TYPE_UI);
+  base::Thread io_thread("IO thread");
+  base::Thread::Options io_thread_options(MessageLoop::TYPE_IO, 0);
+  io_thread.StartWithOptions(io_thread_options);
+
+  net::URLRequestContextGetter* url_request_context_getter_ =

[alexeypa (please no reviews), 2012/10/11 16:57:52]

USe scoped_refptr pointer.

[simonmorris, 2012/10/11 21:27:35]

Done.

+      new remoting::URLRequestContextGetter(
+          message_loop.message_loop_proxy(), io_thread.message_loop_proxy());
+
+  // Run a HostConfigurerWindow.
+  remoting::HostConfigurerWindow host_configurer_window(
+      url_request_context_getter_);
+  host_configurer_window.Create(NULL);
+
+  OleInitialize(NULL);

[alexeypa (please no reviews), 2012/10/11 16:57:52]

You might want to move this call before initialization of
|host_configurer_window| because it might be issuing some COM calls.

[simonmorris, 2012/10/11 21:27:35]

Done.

+  message_loop.Run();

[Sergey Ulanov, 2012/10/11 01:15:24]

base::RunLoop is the preferred way to run MessageLoops now.

[alexeypa (please no reviews), 2012/10/11 16:57:52]

Check out remoting/host/win/elevcated_controller_module.cc. ATL assumes in many
places that you have _AtlModule defined.

[simonmorris, 2012/10/11 21:27:35]

Done.

[simonmorris, 2012/10/11 21:27:35]

Done.

[simonmorris, 2012/10/11 21:27:35]

Done.

+  OleUninitialize();
+}