Don't update jabberId when renaming a host.

remoting/webapp/me2mom/host_list.js

Index: remoting/webapp/me2mom/host_list.js
diff --git a/remoting/webapp/me2mom/host_list.js b/remoting/webapp/me2mom/host_list.js
index 8fd59c0cf9a1f66d12e7c437174382b67bda7633..3e3126b5802c563e07c7f4e576f7cbfdd76c8734 100644
--- a/remoting/webapp/me2mom/host_list.js
+++ b/remoting/webapp/me2mom/host_list.js
@@ -122,8 +122,7 @@ remoting.HostList.prototype.setHosts_ = function(hosts) {
     /** @type {remoting.Host} */
     var host = hosts[i];
     // Validate the entry to make sure it has all the fields we expect.
-    if (host.hostName && host.hostId && host.status && host.jabberId &&

[Jamie, 2011/11/23 20:53:23]

I think we still need to check for jabberId. The purpose of this check is to
make sure that the data we get back from Apiary really does conform to the type
we're expecting. Since we refer to jabberId elsewhere, we need to make sure that
the field exists.

Ideally, this should be a static method on remoting.Host, to keep the check
close to the declaration, but that doesn't really belong in this CL.

[Sergey Ulanov, 2011/11/23 21:10:47]

JabberId may not be set if the host was registered, but never started. Maybe we
should set it to an empty string in this case?

[Sergey Ulanov, 2011/11/23 21:16:50]

Reverted this change for now. Will address in a separate CL

-        host.publicKey) {
+    if (host.hostName && host.hostId && host.status && host.publicKey) {
       var onRename = function() { that.renameHost_(host.hostId); }
       var onDelete = function() { that.deleteHost_(host.hostId); }
       var hostTableEntry = new remoting.HostTableEntry();
@@ -221,7 +220,11 @@ remoting.HostList.prototype.renameHost_ = function(hostId) {
       'Authorization': 'OAuth ' + token,
       'Content-type' : 'application/json; charset=UTF-8'
     };
-    var newHostDetails = { data: hostTableEntry.host };
+    var newHostDetails = { data: {
+      hostId: hostTableEntry.host.hostId,
+      hostName: hostTableEntry.host.hostName,
+      publicKey: hostTableEntry.host.publicKey
+    } };
     remoting.xhr.put(
         'https://www.googleapis.com/chromoting/v1/@me/hosts/' + hostId,
         function(xhr) {},