Automatic host installation for IT2Me on Windows.

remoting/webapp/host_screen.js

 // 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.
 
 /**
  * @fileoverview
  * Functions related to the 'host screen' for Chromoting.
  */
 
 'use strict';
@@ skipping 20 matching lines @@
 
   /** @type {remoting.HostInstallDialog} */
   var hostInstallDialog = null;
 
   var tryInitializeDispatcher = function() {
     hostDispatcher.initialize(createPluginForIt2Me,
                               onDispatcherInitialized,
                               onDispatcherInitializationFailed);
   }
 
- /** @return {remoting.HostPlugin} */
+  /** @return {remoting.HostPlugin} */
   var createPluginForIt2Me = function() {
     return remoting.createNpapiPlugin(
         document.getElementById('host-plugin-container'));
   }
 
+  /** @param {remoting.HostController.AsyncResult} asyncResult */
+  var onHostInstalled = function(asyncResult) {
+    if (asyncResult == remoting.HostController.AsyncResult.OK) {
+      tryInitializeDispatcher();
+    } else if (asyncResult == remoting.HostController.AsyncResult.CANCELLED) {
+      onInstallError(remoting.Error.CANCELLED);

[Sergey Ulanov, 2014/05/21 03:21:59]

Why doesn't HostInstallDialog call onInstallError() directly when installation
was cancelled? It knows that installation failed, so it doesn't look right that
it still calls onHostInstalled(). It also adds dependency on internals of the
HostInstallDialog implementation

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

Good point. Will fix this.

On 2014/05/21 03:21:59, Sergey Ulanov wrote:

+    } else {
+      onInstallError(remoting.Error.UNEXPECTED);
+    }
+  };
+
   var onDispatcherInitialized = function () {
-    remoting.startHostUsingDispatcher_(hostDispatcher);
-  }
+    if (hostDispatcher.usingNpapi()) {

[Sergey Ulanov, 2014/05/21 03:21:59]

nit: This wouldn't be necessary if HostIt2MeDispatcher allowed successful
initialization with the plugin only when the plugin support It2Me and if
HostInstallDialog could instantiate NPAPI plugin directly when necessary (there
is no reason HostIt2MeDispatcher and HostInstallDialog have to use the same
instance of the plugin).

That would make this change a bit simpler, but I understand that this depends on
the direction chosen in the previous CLs, so it's not worth changing now, as we
won't need this logic for too long.

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

Yes this is planned in my next CL. :)

On 2014/05/21 03:21:59, Sergey Ulanov wrote:

+      hostInstallDialog = new remoting.HostInstallDialog();
+      if (navigator.platform == 'Win32') {
+        hostInstallDialog.show(
+            hostDispatcher.getNpapiHost(), onHostInstalled, onInstallError);
+      } else {
+        hostInstallDialog.show(null, onHostInstalled, onInstallError);
+      }
+    } else {
+      // Host alrady installed.
+      remoting.startHostUsingDispatcher_(hostDispatcher);
+    }
+  };
 
   /** @param {remoting.Error} error */
   var onDispatcherInitializationFailed = function(error) {
     if (error != remoting.Error.MISSING_PLUGIN) {
       showShareError_(error);
       return;
     }
 
-    // If we failed to initialize dispatcher then prompt the user to install the
-    // host.
+    // If we failed to initialize the dispatcher then prompt the user to install
+    // the host manually.
     if (hostInstallDialog == null) {
-      var onDone = function(asyncResult) {
-        // TODO (weitaosu): Ignore asyncResult for now because it is not set
-        // during manual host installation. We should fix it after switching
-        // to automatic host installation on Windows.
-        tryInitializeDispatcher();
-      };
-
       hostInstallDialog = new remoting.HostInstallDialog();
 
       (/** @type {remoting.HostInstallDialog} */ hostInstallDialog).show(
-          null,
-          onDone,
-          onInstallPromptError);
+          null, onHostInstalled, onInstallError);
     } else {
       (/** @type {remoting.HostInstallDialog} */ hostInstallDialog).tryAgain();
     }
-  }
+  };
 
   /** @param {remoting.Error} error */
-  var onInstallPromptError = function(error) {
+  var onInstallError = function(error) {
     if (error == remoting.Error.CANCELLED) {
       remoting.setMode(remoting.AppMode.HOME);
     } else {
       showShareError_(error);
     }
   }
 
   tryInitializeDispatcher();
 };
 
@@ skipping 266 matching lines @@
  * @private
  */
 function onNatTraversalPolicyChanged_(enabled) {
   var natBox = document.getElementById('nat-box');
   if (enabled) {
     natBox.classList.add('traversal-enabled');
   } else {
     natBox.classList.remove('traversal-enabled');
   }
 }