Add more host-side connection state logging for IT2Me.

remoting/webapp/crd/js/host_controller.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.
 
 'use strict';
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 /** @constructor */
 remoting.HostController = function() {
-  this.hostDaemonFacade_ = this.createDaemonFacade_();
+  /** @type {remoting.HostDaemonFacade} @private */
+  this.hostDaemonFacade_ = new remoting.HostDaemonFacade();
+
+  /** @param {string} version */
+  var printVersion = function(version) {
+    if (version == '') {
+      console.log('Host not installed.');
+    } else {
+      console.log('Host version: ' + version);
+    }
+  };
+
+  this.getLocalHostVersion().then(printVersion, function() {

[kelvinp, 2015/06/10 18:00:45]

Nit: Use .catch for error handling for readability.

[Jamie, 2015/06/10 18:14:44]

Done.

+    console.log('Host version not available.');
+  });
 };
 
 // The values in the enums below are duplicated in daemon_controller.h except
 // for NOT_INSTALLED.
 /** @enum {number} */
 remoting.HostController.State = {
   NOT_IMPLEMENTED: 0,
   NOT_INSTALLED: 1,
   STOPPED: 2,
   STARTING: 3,
@@ skipping 26 matching lines @@
  * @return {remoting.HostController.AsyncResult} The result enum value.
  */
 remoting.HostController.AsyncResult.fromString = function(result) {
   if (!remoting.HostController.AsyncResult.hasOwnProperty(result)) {
     throw "Invalid HostController.AsyncResult: " + result;
   }
   return remoting.HostController.AsyncResult[result];
 }
 
 /**
- * @return {remoting.HostDaemonFacade}
- * @private
- */
-remoting.HostController.prototype.createDaemonFacade_ = function() {
-  /** @type {remoting.HostDaemonFacade} @private */
-  var hostDaemonFacade = new remoting.HostDaemonFacade();
-
-  /** @param {string} version */
-  var printVersion = function(version) {
-    if (version == '') {
-      console.log('Host not installed.');
-    } else {
-      console.log('Host version: ' + version);
-    }
-  };
-
-  hostDaemonFacade.getDaemonVersion().then(printVersion, function() {
-    console.log('Host version not available.');
-  });
-
-  return hostDaemonFacade;
-};
-
-/**
  * Set of features for which hasFeature() can be used to test.
  *
  * @enum {string}
  */
 remoting.HostController.Feature = {
   PAIRING_REGISTRY: 'pairingRegistry',
   OAUTH_CLIENT: 'oauthClient'
 };
 
 /**
@@ skipping 319 matching lines @@
     }
     onDone(hostId);
   };
 
   this.hostDaemonFacade_.getDaemonConfig().then(onConfig, function(error) {
     onDone(null);
   });
 };
 
 /**
+ * @return {Promise<string>} Promise that resolves with the host version, if
+ *     installed, or rejects otherwise.
+ */
+remoting.HostController.prototype.getLocalHostVersion = function() {
+  return this.hostDaemonFacade_.getDaemonVersion();
+};
+
+/**
  * Fetch the list of paired clients for this host.
  *
  * @param {function(Array<remoting.PairedClient>):void} onDone
  * @param {function(!remoting.Error):void} onError
  * @return {void}
  */
 remoting.HostController.prototype.getPairedClients = function(onDone,
                                                               onError) {
   this.hostDaemonFacade_.getPairedClients().then(
       onDone, remoting.Error.handler(onError));
@@ skipping 68 matching lines @@
     emailPromise.then(function(/** string */ email) {
       signalStrategy.connect(remoting.settings.XMPP_SERVER, email, token);
     });
   });
 
   return result;
 };
 
 /** @type {remoting.HostController} */
 remoting.hostController = null;