Added ability to register new hosts using GCD.

remoting/webapp/crd/js/host_list_api_impl.js

 // Copyright 2014 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
  * REST API for host-list management.
  */
 
 /** @suppress {duplicate} */
@@ skipping 24 matching lines @@
     url: remoting.settings.DIRECTORY_API_BASE_URL + '/@me/hosts',
     urlParams: {
       hostClientId: hostClientId
     },
     jsonContent: newHostDetails,
     acceptJson: true,
     useIdentity: true
   }).start().then(function(response) {
     if (response.status == 200) {
       var result = response.getJson();
+      var authCode = '';
       if (result['data']) {
-        return base.getStringAttr(result['data'], 'authorizationCode', '');
-      } else {
-        return '';
+        authCode = base.getStringAttr(result['data'], 'authorizationCode', '');
       }
+      return { authCode: authCode, email: '' };

[Jamie, 2015/04/27 23:24:24]

Rather than adding code at call sites to deal with there being no email, why not
fill it in here?

[John Williams, 2015/04/28 20:27:40]

The call site already has to fetch the user's email address, and filling in a
missing email address is logically part of building the host config and not part
of registering the host.  I don't want to clutter this method with things that
don't involve communicating with the registry.

     } else {
       console.log(
           'Failed to register the host. Status: ' + response.status +
           ' response: ' + response.getText());
       throw new remoting.Error(remoting.Error.Tag.REGISTRATION_FAILED);
     }
   });
 };
 
 /** @override */
@@ skipping 93 matching lines @@
     if (opt_ignoreErrors && error.hasTag.apply(error, opt_ignoreErrors)) {
       return;
     }
 
     throw error;
   };
   return result;
 };
 
 })();