Automatic host installation for IT2Me on Windows.

remoting/webapp/host_dispatcher.js

Index: remoting/webapp/host_dispatcher.js
diff --git a/remoting/webapp/host_dispatcher.js b/remoting/webapp/host_dispatcher.js
index a5ea504ecefeb75345cae26f972260e61cfc49e9..fe20019af0cb9e2868c06d4f263e68100bf69d0e 100644
--- a/remoting/webapp/host_dispatcher.js
+++ b/remoting/webapp/host_dispatcher.js
@@ -87,6 +87,13 @@ remoting.HostDispatcher.prototype.tryToInitialize_ = function() {
 };
 
 /**
+ * @return {remoting.HostPlugin}
+ */
+remoting.HostDispatcher.prototype.getNpapiHost = function() {

[Jamie, 2014/05/20 21:43:06]

I don't like the host dispatcher exposing one of its underlying interfaces in
this way, but if it's going to be short-lived, I can live it for now :)

[weitao, 2014/05/20 22:09:13]

I do plan to remove this in the subsequent CL's when I remove the dispatching
code.

On 2014/05/20 21:43:06, Jamie wrote:

+  return this.npapiHost_;
+}
+
+/**
  * @param {remoting.HostController.Feature} feature The feature to test for.
  * @param {function(boolean):void} onDone
  * @return {void}
@@ -387,6 +394,9 @@ remoting.HostDispatcher.prototype.startDaemon =
 };
 
 /**
+ * This function installs the chromoting host using the NPAPI plugin and should

[Jamie, 2014/05/20 21:43:06]

Nit: We shouldn't refer to "chromoting" in our source. You can just use "host".

Although it doesn't look like the right comment anyway.

[weitao, 2014/05/20 22:09:13]

Done.

+ * only be called on Windows.
+ *
  * @param {function(remoting.HostController.AsyncResult):void} onDone
  * @param {function(remoting.Error):void} onError
  * @return {void}