[Chromoting] Move app-specific code out of remoting.js

remoting/webapp/crd/js/host_list.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.
 
 /**
  * @fileoverview
  * Class representing the host-list portion of the home screen UI.
  */
 
 'use strict';
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 /**
  * Create a host list consisting of the specified HTML elements, which should
  * have a common parent that contains only host-list UI as it will be hidden
  * if the host-list is empty.
  *
  * @constructor
  * @param {Element} table The HTML <div> to contain host-list.
  * @param {Element} noHosts The HTML <div> containing the "no hosts" message.
  * @param {Element} errorMsg The HTML <div> to display error messages.
  * @param {Element} errorButton The HTML <button> to display the error
  *     resolution action.
  * @param {HTMLElement} loadingIndicator The HTML <span> to update while the
  *     host list is being loaded. The first element of this span should be
  *     the reload button.
+ * @param {function(!remoting.Error)} onError Function to call when an error
+ *     occurs.
  */
 remoting.HostList = function(table, noHosts, errorMsg, errorButton,
-                             loadingIndicator) {
+                             loadingIndicator, onError) {
   /** @private {Element} */
   this.table_ = table;
   /**
    * TODO(jamiewalch): This should be doable using CSS's sibling selector,
    * but it doesn't work right now (crbug.com/135050).
    * @private {Element}
    */
   this.noHosts_ = noHosts;
   /** @private {Element} */
   this.errorMsg_ = errorMsg;
   /** @private {Element} */
   this.errorButton_ = errorButton;
   /** @private {HTMLElement} */
   this.loadingIndicator_ = loadingIndicator;
+  this.onError_ = onError;
+
   /** @private {Array<remoting.HostTableEntry>} */
   this.hostTableEntries_ = [];
   /** @private {Array<remoting.Host>} */
   this.hosts_ = [];
   /** @private {!remoting.Error} */
   this.lastError_ = remoting.Error.none();
   /** @private {remoting.LocalHostSection} */
   this.localHostSection_ = new remoting.LocalHostSection(
       /** @type {HTMLElement} */ (document.querySelector('.daemon-control')),
       new remoting.LocalHostSection.Controller(
-          this, new remoting.HostSetupDialog(remoting.hostController)));
+          this,
+          new remoting.HostSetupDialog(remoting.hostController, onError)));
 
   /** @private {number} */
   this.webappMajorVersion_ = parseInt(chrome.runtime.getManifest().version, 10);
 
   this.errorButton_.addEventListener('click',
                                      this.onErrorClick_.bind(this),
                                      false);
   var reloadButton = this.loadingIndicator_.firstElementChild;
   /** @type {remoting.HostList} */
   var that = this;
@@ skipping 221 matching lines @@
  * @return {void} Nothing.
  * @private
  */
 remoting.HostList.prototype.deleteHost_ = function(hostTableEntry) {
   this.table_.removeChild(hostTableEntry.element());
   var index = this.hostTableEntries_.indexOf(hostTableEntry);
   if (index != -1) {
     this.hostTableEntries_.splice(index, 1);
   }
   remoting.hostListApi.remove(hostTableEntry.host.hostId, base.doNothing,
-                              remoting.showErrorMessage);
+                              this.onError_);
 };
 
 /**
  * Prepare a host for renaming by replacing its name with an edit box.
  * @param {remoting.HostTableEntry} hostTableEntry The host to be renamed.
  * @return {void} Nothing.
  */
 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;
     }
   }
   this.save_();
 
   remoting.hostListApi.put(hostTableEntry.host.hostId,
                            hostTableEntry.host.hostName,
                            hostTableEntry.host.publicKey,
                            function() {},
-                           remoting.showErrorMessage);
+                           this.onError_);
 };
 
 /**
  * Unregister a host.
  * @param {string} hostId The id of the host to be removed.
  * @param {function(void)=} opt_onDone
  * @return {void} Nothing.
  */
 remoting.HostList.prototype.unregisterHostById = function(hostId, opt_onDone) {
   var that = this;
   var onDone = opt_onDone || base.doNothing;
 
   var host = this.getHostForId(hostId);
   if (!host) {
     console.log('Skipping host un-registration as the host is not registered ' +
                 'under the current account');
     onDone();
     return;
   }
 
   var onRemoved = function() {
     that.refresh(function() {
       that.display();
       onDone();
     });
   };
-  remoting.hostListApi.remove(hostId, onRemoved, remoting.showErrorMessage);
+  remoting.hostListApi.remove(hostId, onRemoved, this.onError_);
 };
 
 /**
  * Set the state of the local host and localHostId if any.
  *
  * @param {remoting.HostController.State} state State of the local host.
  * @param {string?} hostId ID of the local host, or null.
  * @return {void} Nothing.
  */
 remoting.HostList.prototype.setLocalHostStateAndId = function(state, hostId) {
@@ skipping 58 matching lines @@
   }
 };
 
 /**
  * Key name under which Me2Me hosts are cached.
  */
 remoting.HostList.HOSTS_KEY = 'me2me-cached-hosts';
 
 /** @type {remoting.HostList} */
 remoting.hostList = null;