[Chromoting] Make the Windows host controller ask the user to confirm host registration.

remoting/host/elevated_controller_module_win.cc

Index: remoting/host/elevated_controller_module_win.cc
diff --git a/remoting/host/elevated_controller_module_win.cc b/remoting/host/elevated_controller_module_win.cc
index 9d6442b14b0634534411efcca97d5b407e7cf3aa..38bbc634555538962f1f910dec03387ceb896a52 100644
--- a/remoting/host/elevated_controller_module_win.cc
+++ b/remoting/host/elevated_controller_module_win.cc
@@ -28,9 +28,13 @@ class ElevatedControllerModuleWin
 
 remoting::ElevatedControllerModuleWin _AtlModule;
 
-int WINAPI WinMain(HINSTANCE, HINSTANCE, LPSTR, int command) {
+HINSTANCE g_hInstance = NULL;

[alexeypa (please no reviews), 2012/04/12 21:20:41]

nit: g_hInstance -> g_instance

[alexeypa (please no reviews), 2012/04/12 21:20:41]

I believe you should be able to use _AtlModule.GetModuleInstance() since it is
an ATL-based applications.

[simonmorris, 2012/04/12 22:11:44]

Done.

[simonmorris, 2012/04/12 22:11:44]

This way is consistent with host_plugin.cc. Is there a benefit to doing it as
you suggest?

[alexeypa (please no reviews), 2012/04/12 23:10:33]

This way you don't pollute the global namespace and it is consistent with other
ATL code (since this is an ATL application)

[simonmorris, 2012/04/13 00:11:50]

Done.

+
+int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE, LPSTR, int command) {

[alexeypa (please no reviews), 2012/04/12 21:20:41]

nit: hInstance -> instance

[simonmorris, 2012/04/12 22:11:44]

Done.

   CommandLine::Init(0, NULL);
 
+  g_hInstance = hInstance;
+
   // This object instance is required by Chrome code (for example,
   // FilePath, LazyInstance, MessageLoop).
   base::AtExitManager exit_manager;