Update Chromoting to use /third_party/closure_compiler.

remoting/webapp/crd/js/host_daemon_facade.js

Index: remoting/webapp/crd/js/host_daemon_facade.js
diff --git a/remoting/webapp/crd/js/host_daemon_facade.js b/remoting/webapp/crd/js/host_daemon_facade.js
index f77504408f42f884810283670457f2b973cdfa70..de5e335e1acbe8753111e60cb78ba436c7846c37 100644
--- a/remoting/webapp/crd/js/host_daemon_facade.js
+++ b/remoting/webapp/crd/js/host_daemon_facade.js
@@ -91,26 +91,25 @@ remoting.HostDaemonFacade.prototype.initialize_ = function() {
  * @private
  */
 remoting.HostDaemonFacade.prototype.connectNative_ = function() {
-  return new Promise(
-    /**
-     * @param {function(*=):void} resolve
-     * @param {function(*=):void} reject
-     * @this {remoting.HostDaemonFacade}
-     */
-    function(resolve, reject) {
-      try {
-        this.port_ = chrome.runtime.connectNative(
-            'com.google.chrome.remote_desktop');
-        this.port_.onMessage.addListener(this.onIncomingMessageCallback_);
-        this.port_.onDisconnect.addListener(this.onDisconnectCallback_);
-        this.postMessageInternal_({type: 'hello'}, resolve, reject);
-      } catch (err) {
-        console.log('Native Messaging initialization failed: ',
-                    /** @type {*} */ (err));
-        reject();
-      }
-    }.bind(this)
-  );
+  /**
+   * @this {remoting.HostDaemonFacade}
+   * @param {function(?):void} resolve
+   * @param {function(*):void} reject
+   */
+  var connect = function(resolve, reject) {
+    try {
+      this.port_ = chrome.runtime.connectNative(
+          'com.google.chrome.remote_desktop');
+      this.port_.onMessage.addListener(this.onIncomingMessageCallback_);
+      this.port_.onDisconnect.addListener(this.onDisconnectCallback_);
+      this.postMessageInternal_({type: 'hello'}, resolve, reject);
+    } catch (/** @type {*} */ err) {
+      console.log('Native Messaging initialization failed: ', err);
+      reject(false);

[kelvinp, 2015/01/12 22:51:28]

why false here?

[garykac, 2015/01/13 00:28:39]

reject() requires at least 1 argument(s) and no more than 1 argument(s)

The value 'false' is arbitrary, but it seemed better than true. Is something
like '0' better here?

[Tyler Breisacher (Chromium), 2015/01/13 03:01:00]

We actually changed this in
https://github.com/google/closure-compiler/commit/921c464b40f3068f17c4c8ab017...
so you could probably remove the 'false' (or null or whatever you decide to
pass) after the next time you bump the compiler version.

+    }
+  };
+
+  return new Promise(connect.bind(this));
 };
 
 /**
@@ -213,9 +212,8 @@ remoting.HostDaemonFacade.prototype.onIncomingMessage_ = function(message) {
     }
 
     this.handleIncomingMessage_(message, reply.onDone);
-  } catch (e) {
-    console.error('Error while processing native message' +
-                  /** @type {*} */ (e));
+  } catch (/** @type {*} */ e) {
+    console.error('Error while processing native message' + e);
     reply.onError(remoting.Error.UNEXPECTED);
   }
 }
@@ -340,7 +338,8 @@ remoting.HostDaemonFacade.prototype.onDisconnect_ = function() {
   var pendingReplies = this.pendingReplies_;
   this.pendingReplies_ = {};
   for (var id in pendingReplies) {
-    pendingReplies[/** @type {number} */(id)].onError(this.error_);
+    var num_id = parseInt(id, 10);
+    pendingReplies[num_id].onError(this.error_);
   }
 }