Send enums as strings between Chromoting Native Messaging host and web-app.

remoting/webapp/host_native_messaging.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
  * Class to communicate with the Host components via Native Messaging.
  */
 
 'use strict';
@@ skipping 91 matching lines @@
 }
 
 /**
  * Returns |result| as an AsyncResult. If |result| is not valid, returns null
  * and logs an error.
  *
  * @param {*} result
  * @return {remoting.HostController.AsyncResult?} Converted result.
  */
 function asAsyncResult_(result) {
-  if (!checkType_('result', result, 'number')) {
+  if (!checkType_('result', result, 'string')) {
     return null;
   }
-  for (var i in remoting.HostController.AsyncResult) {
-    if (remoting.HostController.AsyncResult[i] == result) {
-      return remoting.HostController.AsyncResult[i];
-    }
+  if (!remoting.HostController.AsyncResult.hasOwnProperty(result)) {
+    console.error('NativeMessaging: unexpected result code: ', result);
+    return null;
   }
-  console.error('NativeMessaging: unexpected result code: ', result);
-  return null;
+  return remoting.HostController.AsyncResult[result];
 }
 
 /**
  * Returns |result| as a HostController.State. If |result| is not valid,
  * returns null and logs an error.
  *
  * @param {*} result
  * @return {remoting.HostController.State?} Converted result.
  */
 function asHostState_(result) {
-  if (!checkType_('result', result, 'number')) {
+  if (!checkType_('result', result, 'string')) {
     return null;
   }
-  for (var i in remoting.HostController.State) {
-    if (remoting.HostController.State[i] == result) {
-      return remoting.HostController.State[i];
-    }
+  if (!remoting.HostController.State.hasOwnProperty(result)) {
+    console.error('NativeMessaging: unexpected result code: ', result);
+    return null;
   }
-  console.error('NativeMessaging: unexpected result code: ', result);
-  return null;
+  return remoting.HostController.State[result];
 }
 
 /**
  * Attaches a new ID to the supplied message, and posts it to the Native
  * Messaging port, adding |callback| to the list of pending replies.
  * |message| should have its 'type' field set, and any other fields set
  * depending on the message type.
  *
  * @param {{type: string}} message The message to post.
  * @param {?function(...):void} callback The callback, if any, to be triggered
@@ skipping 301 matching lines @@
 
 /**
  * Gets the installed/running state of the Host process.
  *
  * @param {function(remoting.HostController.State):void} callback Callback.
  * @return {void} Nothing.
  */
 remoting.HostNativeMessaging.prototype.getDaemonState = function(callback) {
   this.postMessage_({type: 'getDaemonState'}, callback);
 }