Don't update jabberId when renaming a host.

remoting/webapp/me2mom/host_list.js

 // Copyright (c) 2011 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 representing the host-list portion of the home screen UI.
  */
 
 'use strict';
@@ skipping 104 matching lines @@
   this.table_.innerHTML = '';
   this.showError_(null);
   this.hostTableEntries_ = [];
 
   /** @type {remoting.HostList} */
   var that = this;
   for (var i = 0; i < hosts.length; ++i) {
     /** @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();
       hostTableEntry.init(host, onRename, onDelete);
       this.hostTableEntries_[i] = hostTableEntry;
       this.table_.appendChild(hostTableEntry.tableRow);
     }
   }
 
   this.showOrHide_(this.hostTableEntries_.length != 0);
@@ skipping 77 matching lines @@
   if (!hostTableEntry) {
     console.error('No host registered for id ' + hostId);
     return;
   }
   /** @param {string} token */
   var renameHost = function(token) {
     var headers = {
       '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) {},
         JSON.stringify(newHostDetails),
         headers);
   }
   remoting.oauth2.callWithToken(renameHost);
 };
 
 /**
  * Class name for the host list when it is collapsed.
  * @private
  */
 remoting.HostList.COLLAPSED_ = 'collapsed';
 
 /** @type {remoting.HostList} */
 remoting.hostList = null;