Automatic host installation for IT2Me on Windows.

remoting/webapp/host_install_dialog.js

Index: remoting/webapp/host_install_dialog.js
diff --git a/remoting/webapp/host_install_dialog.js b/remoting/webapp/host_install_dialog.js
index 71bf84dc30c6908f11ee564c1c1953eb0d86ef79..448b2d7bfe26681065d9ac379dd486181aee9d23 100644
--- a/remoting/webapp/host_install_dialog.js
+++ b/remoting/webapp/host_install_dialog.js
@@ -49,8 +49,7 @@ remoting.HostInstallDialog.hostDownloadUrls = {
 /**
  * Starts downloading host components and shows installation prompt.
  *
- * @param {remoting.HostController} hostController Used to install the host on
- *     Windows.
+ * @param {remoting.HostPlugin} hostPlugin Used to install the host on Windows.
  * @param {function(remoting.HostController.AsyncResult):void} onDone Callback
  *     called when user clicks Ok, presumably after installing the host. The
  *     handler must verify that the host has been installed and call tryAgain()
@@ -60,14 +59,24 @@ remoting.HostInstallDialog.hostDownloadUrls = {
  * @return {void}
  */
 remoting.HostInstallDialog.prototype.show = function(
-    hostController, onDone, onError) {
+    hostPlugin, onDone, onError) {
   // On Windows, host installation is automatic (handled by the NPAPI plugin)
   // and we don't show the dialog. On Mac and Linux, we show the dialog and the
   // user is expected to manually install the host before clicking OK.
   // TODO (weitaosu): Make host installation automatic for IT2Me (like Me2Me) on
   // Windows. Currently hostController is always null for IT2Me.
-  if (navigator.platform == 'Win32' && hostController != null) {
-    hostController.installHost(onDone, onError);
+  if (navigator.platform == 'Win32' && hostPlugin != null) {
+    // Currently we show two dialogs (each with a UAC prompt) when a user
+    // enables the host for the first time, one for installing the host (by the
+    // plugin) and the other for starting the host (by the native messaging
+    // host). We'd like to reduce it to one but don't have a good solution
+    // right now.
+    // We also show the same message on the two dialogs because. We don't want
+    // to confuse the user by saying "Installing Remote Desktop" because in
+    // their mind "Remote Desktop" (the webapp) has already been installed.
+    remoting.showSetupProcessingMessage(/*i18n-content*/'HOST_SETUP_STARTING');
+
+    hostPlugin.installHost(onDone);

[Sergey Ulanov, 2014/05/21 16:07:58]

This just starts installing the host silently. I past we discussed prompting the
user first, at least for It2Me. Is that still the plan?

[weitao, 2014/05/21 18:04:22]

You are right! I had forgot about this after redoing the IT2Me work (last and
this CL) and the two weeks absence. I will fix this.

On 2014/05/21 16:07:58, Sergey Ulanov wrote:

   } else {
     this.continueInstallButton_.addEventListener(
         'click', this.onOkClickedHandler_, false);