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

remoting/webapp/crd/js/crd_main.js

 // Copyright 2014 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 || {};
 
 /**
  * Initialize the host list.
  */
 remoting.initHostlist_ = function() {
   remoting.hostController = new remoting.HostController();
   remoting.hostList = new remoting.HostList(
       document.getElementById('host-list'),
       document.getElementById('host-list-empty'),
       document.getElementById('host-list-error-message'),
       document.getElementById('host-list-refresh-failed-button'),
-      document.getElementById('host-list-loading-indicator'));
+      document.getElementById('host-list-loading-indicator'),
+      remoting.app.onAuthError.bind(remoting.app));

[kelvinp, 2015/03/19 17:50:20]

This is a bit misleading as the error that the host list reports are not auth
issues but errors in deleting/renaming/unregistering the host.

What about keeping remoting.showErrorMessage() but moving it to
crd_events_handlers.js or crd_main.js?

[garykac, 2015/03/19 21:58:38]

Done.

 
   isHostModeSupported_().then(
       /** @param {Boolean} supported */
       function(supported) {
         if (supported) {
           var noShare = document.getElementById('unsupported-platform-message');
           noShare.parentNode.removeChild(noShare);
         } else {
           var button = document.getElementById('share-button');
           button.disabled = true;
@@ skipping 150 matching lines @@
 }
 
 
 remoting.startDesktopRemoting = function() {
   remoting.app = new remoting.Application(remoting.app_capabilities());
   var desktop_remoting = new remoting.DesktopRemoting(remoting.app);
   remoting.app.start();
 };
 
 window.addEventListener('load', remoting.startDesktopRemotingForTesting, false);