Eliminated named constants for instances of remoting.Error.

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 43aec855d393d963f0100fd079057c2eb7c74274..eab06de2d7c4ca8ca5c0a8851fecd8ff7e0bf8a3 100644
--- a/remoting/webapp/crd/js/host_daemon_facade.js
+++ b/remoting/webapp/crd/js/host_daemon_facade.js
@@ -42,7 +42,7 @@ remoting.HostDaemonFacade = function() {
   this.initializingPromise_ = null;
 
   /** @private {!remoting.Error} */
-  this.error_ = remoting.Error.NONE;
+  this.error_ = remoting.Error.none();
 
   /** @private */
   this.onIncomingMessageCallback_ = this.onIncomingMessage_.bind(this);
@@ -206,8 +206,8 @@ remoting.HostDaemonFacade.prototype.onIncomingMessage_ = function(message) {
 
     this.handleIncomingMessage_(message, reply.onDone);
   } catch (/** @type {*} */ e) {
-    console.error('Error while processing native message' + e);
-    reply.onError(remoting.Error.UNEXPECTED);
+    console.error('Error while processing native message', e);

[Jamie, 2015/03/13 20:26:01]

No change required: This comma is correct, I think, since we don't know the type
of e.

+    reply.onError(remoting.Error.unexpected());
   }
 }
 
@@ -324,8 +324,9 @@ remoting.HostDaemonFacade.prototype.onDisconnect_ = function() {
 
   // If initialization hasn't finished then assume that the port was
   // disconnected because Native Messaging host is not installed.
-  this.error_ = this.initializingPromise_ ? remoting.Error.MISSING_PLUGIN :
-                                            remoting.Error.UNEXPECTED;
+  this.error_ = this.initializingPromise_ ?
+      new remoting.Error(remoting.Error.Tag.MISSING_PLUGIN) :
+      remoting.Error.unexpected();
 
   // Notify the error-handlers of any requests that are still outstanding.
   var pendingReplies = this.pendingReplies_;