Added better error and OAuth support in xhr.js.

remoting/webapp/crd/js/host_list_api_impl.js

Index: remoting/webapp/crd/js/host_list_api_impl.js
diff --git a/remoting/webapp/crd/js/host_list_api_impl.js b/remoting/webapp/crd/js/host_list_api_impl.js
index 52f9377e58a12aa64aa256b4094b69d3cc5397b7..99007defdccf20312cb4f3e2c802438c7813ce11 100644
--- a/remoting/webapp/crd/js/host_list_api_impl.js
+++ b/remoting/webapp/crd/js/host_list_api_impl.js
@@ -67,7 +67,7 @@ remoting.HostListApiImpl.prototype.put =
       url: remoting.settings.DIRECTORY_API_BASE_URL + '/@me/hosts/' + hostId,
       jsonContent: newHostDetails,
       oauthToken: token
-    }).start().then(remoting.Xhr.defaultResponse(onDone, onError));
+    }).start().then(remoting.HostListApiImpl.defaultResponse_(onDone, onError));
   };
   remoting.identity.getToken().then(onToken, remoting.Error.handler(onError));

[Jamie, 2015/03/24 19:44:11]

Why have you reverted the useIdentity change as well?

[John Williams, 2015/03/24 23:34:41]

Accident.

 };
@@ -86,7 +86,7 @@ remoting.HostListApiImpl.prototype.remove = function(hostId, onDone, onError) {
       method: 'DELETE',
       url: remoting.settings.DIRECTORY_API_BASE_URL + '/@me/hosts/' + hostId,
       oauthToken: token
-    }).start().then(remoting.Xhr.defaultResponse(
+    }).start().then(remoting.HostListApiImpl.defaultResponse_(
         onDone, onError, [remoting.Error.Tag.NOT_FOUND]));
   };
   remoting.identity.getToken().then(onToken, remoting.Error.handler(onError));
@@ -135,6 +135,35 @@ remoting.HostListApiImpl.prototype.parseHostListResponse_ =
   }
 };
 
+/**
+ * Generic success/failure response proxy.
+ *
+ * @param {function():void} onDone
+ * @param {function(!remoting.Error):void} onError
+ * @param {Array<remoting.Error.Tag>=} opt_ignoreErrors
+ * @return {function(!remoting.Xhr.Response):void}
+ * @private
+ */
+remoting.HostListApiImpl.defaultResponse_ = function(
+    onDone, onError, opt_ignoreErrors) {
+  /** @param {!remoting.Xhr.Response} response */
+  var result = function(response) {
+    var error = remoting.Error.fromHttpStatus(response.status);
+    if (error.isNone()) {
+      onDone();
+      return;
+    }
+
+    if (opt_ignoreErrors && error.hasTag.apply(error, opt_ignoreErrors)) {
+      onDone();
+      return;
+    }
+
+    onError(error);
+  };
+  return result;
+};
+
 /** @type {remoting.HostListApi} */
 remoting.hostListApi = new remoting.HostListApiImpl();