Don't register more than one 'connect' event handler per HostTableEntry. Don't create more than one…

remoting/webapp/host_controller.js

 // Copyright (c) 2012 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.
 
 'use strict';
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 /** @constructor */
@@ skipping 261 matching lines @@
   this.localHost = host;
   this.setTooltips();
   /** @type {remoting.HostController} */
   var that = this;
   if (host) {
     /** @param {remoting.HostTableEntry} host */
     var renameHost = function(host) {
       remoting.hostList.renameHost(host);
       that.setTooltips();
     };
-    /** @type {remoting.HostTableEntry} @private */
-    this.hostTableEntry_ = new remoting.HostTableEntry();
-    this.hostTableEntry_.init(host,
-                              document.getElementById('this-host-connect'),
-                              document.getElementById('this-host-name'),
-                              document.getElementById('this-host-rename'),
-                              renameHost);
+    if (!this.hostTableEntry_) {
+      /** @type {remoting.HostTableEntry} @private */
+      this.hostTableEntry_ = new remoting.HostTableEntry();
+      this.hostTableEntry_.init(host,
+                                document.getElementById('this-host-connect'),
+                                document.getElementById('this-host-name'),
+                                document.getElementById('this-host-rename'),
+                                renameHost);
+    } else {
+      // TODO(jamiewalch): This is hack to prevent multiple click handlers being
+      // registered for the same DOM elements if this method is called more than
+      // once. A better solution would be to let HostTable create the daemon row
+      // like it creates the rows for non-local hosts.
+      this.hostTableEntry_.host = host;
+    }
   } else {
     this.hostTableEntry_ = null;
   }
 };
 
 /**
  * Update the internal state so that the local host can be correctly filtered
  * out of the host list.
  *
  * @param {remoting.HostList} hostList The new host list, returned by the
  *     Chromoting service.
  * @param {function():void} onDone Completion callback. TODO(jamiewalch): For
  *     now, this is synchronous and reads the host id from local storage. In
  *     the future, it will asynchronously read the host id from the plugin
  *     (crbug.com/121518).
  */
 remoting.HostController.prototype.onHostListRefresh =
     function(hostList, onDone) {
   var hostId = window.localStorage.getItem('me2me-host-id');
   if (hostId && typeof(hostId) == 'string') {
     this.setHost(hostList.getHostForId(/** @type{string} */(hostId)));
   } else {
     this.setHost(null);
   }
   onDone();
 };
 
 /** @type {remoting.HostController} */
 remoting.hostController = null;