Automatic host installation for IT2Me on Windows.

remoting/webapp/host_dispatcher.js

 // Copyright 2013 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
  * This class provides an interface between the HostController and either the
  * NativeMessaging Host or the Host NPAPI plugin, depending on whether or not
  * NativeMessaging is supported. Since the test for NativeMessaging support is
  * asynchronous, this class stores any requests on a queue, pending the result
@@ skipping 69 matching lines @@
     that.state_ = that.npapiHost_ ? remoting.HostDispatcher.State.NPAPI
                                   : remoting.HostDispatcher.State.NOT_INSTALLED;
     sendPendingRequests();
   }
 
   this.nativeMessagingHost_.initialize(onNativeMessagingInit,
                                        onNativeMessagingFailed);
 };
 
 /**
+ * @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}
  */
 remoting.HostDispatcher.prototype.hasFeature = function(
     feature, onDone) {
   switch (this.state_) {
     case remoting.HostDispatcher.State.UNKNOWN:
       this.pendingRequests_.push(
           this.hasFeature.bind(this, feature, onDone));
@@ skipping 280 matching lines @@
         onError(remoting.Error.MISSING_PLUGIN);
       }
       break;
     case remoting.HostDispatcher.State.NOT_INSTALLED:
       onError(remoting.Error.MISSING_PLUGIN);
       break;
   }
 };
 
 /**
+ * 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}
  */
 remoting.HostDispatcher.prototype.stopDaemon = function(onDone, onError) {
   switch (this.state_) {
     case remoting.HostDispatcher.State.UNKNOWN:
       this.pendingRequests_.push(this.stopDaemon.bind(this, onDone, onError));
       break;
     case remoting.HostDispatcher.State.NATIVE_MESSAGING:
@@ skipping 236 matching lines @@
   }
 };
 
 /**
  * Returns true if the NPAPI plugin is being used.
  * @return {boolean}
  */
 remoting.HostDispatcher.prototype.usingNpapiPlugin = function() {
   return this.state_ == remoting.HostDispatcher.State.NPAPI;
 }