[Chromoting] The webapp handles host renaming correctly.

BUG=113763
TEST=Rename a host. Check the hover text for that host. Connect to the host, and check the title of the resulting page.

Review URL: https://chromiumcodereview.appspot.com/9722014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@127476 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 13 insertions, 0 deletions

diff --git a/remoting/webapp/host_list.js b/remoting/webapp/host_list.js
index 69f49a5..e7d813c 100644
--- a/remoting/webapp/host_list.js
+++ b/remoting/webapp/host_list.js
@@ -249,6 +249,15 @@ remoting.HostList.prototype.deleteHost_ = function(hostTableEntry) {
  * @private
  */
 remoting.HostList.prototype.renameHost_ = function(hostTableEntry) {
+  for (var i = 0; i < this.hosts_.length; ++i) {
+    if (this.hosts_[i].hostId == hostTableEntry.host.hostId) {
+      this.hosts_[i].hostName = hostTableEntry.host.hostName;
+      break;
+    }
+  }
+  window.localStorage.setItem(remoting.HostList.HOSTS_KEY,
+                              JSON.stringify(this.hosts_));
+
   /** @param {string} token */
   var renameHost = function(token) {
     var headers = {
diff --git a/remoting/webapp/host_table_entry.js b/remoting/webapp/host_table_entry.js
index 5756fd2..7132fd9 100644
--- a/remoting/webapp/host_table_entry.js
+++ b/remoting/webapp/host_table_entry.js
@@ -160,6 +160,10 @@ remoting.HostTableEntry.prototype.commitRename_ = function() {
   if (editBox) {
     if (this.host.hostName != editBox.value) {
       this.host.hostName = editBox.value;
+      if (this.host.status == 'ONLINE') {
+        this.tableRow.title = chrome.i18n.getMessage(
+            /*i18n-content*/'TOOLTIP_CONNECT', this.host.hostName);
+      }
       this.onRename_(this);
     }
     this.removeEditBox_();