Remove 'Enable' button on unsupported platforms. Added 'No hosts' message.

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 62 matching lines @@
   this.plugin_.getUsageStatsConsent(callback);
 };
 
 /**
  * Show or hide daemon-specific parts of the UI.
  * @return {void} Nothing.
  */
 remoting.HostController.prototype.updateDom = function() {
   var match = '';
   var state = this.state();
-  switch (state) {
-    case remoting.HostController.State.STARTED:
-      remoting.updateModalUi('enabled', 'data-daemon-state');
-      break;
-    case remoting.HostController.State.NOT_IMPLEMENTED:
-      document.getElementById('start-daemon').disabled = true;
-      document.getElementById('start-daemon-message').innerText =
-          chrome.i18n.getMessage(
-              /*i18n-content*/'HOME_DAEMON_DISABLED_MESSAGE');
-      // No break;
-    case remoting.HostController.State.STOPPED:
-    case remoting.HostController.State.NOT_INSTALLED:
-      remoting.updateModalUi('disabled', 'data-daemon-state');
-      break;
-  }
+  var enabled = (state == remoting.HostController.State.STARTED);
+  var supported = (state != remoting.HostController.State.NOT_IMPLEMENTED);
+  remoting.updateModalUi(enabled ? 'enabled' : 'disabled', 'data-daemon-state');
+  document.getElementById('daemon-control').hidden = !supported;
+  document.getElementById('host-list-empty-instructions').hidden = supported;
 };
 
 /**
  * Set tool-tips for the 'connect' action. We can't just set this on the
  * parent element because the button has no tool-tip, and therefore would
  * inherit connectStr.
  *
  * return {void} Nothing.
  */
 remoting.HostController.prototype.setTooltips = function() {
@@ skipping 159 matching lines @@
  * @param {string} configStr The host configuration, JSON encoded to a string.
  * @return {Object.<string,string>|null} The host configuration.
  */
 function parseHostConfig_(configStr) {
   var config = /** @type {Object.<string,string>} */ jsonParseSafe(configStr);
   if (config &&
       typeof config['host_id'] == 'string' &&
       typeof config['xmpp_login'] == 'string') {
     return config;
   } else {
     if (configStr != '{}') {

[Wez, 2012/06/29 21:40:51]

Under what circumstances do we see empty host configs?

[Jamie, 2012/06/29 21:56:27]

If there's no host registered you get this.

       console.error('Invalid getDaemonConfig response.');
     }
   }
   return null;
 }
 
 /**
  * @param {string} newPin The new PIN to set
  * @param {function(remoting.HostController.AsyncResult):void} callback
  *     Callback to be called when finished.
@@ skipping 86 matching lines @@
   try {
     this.plugin_.getDaemonConfig(onConfig);
   } catch (err) {
     this.setHost(null);
     onDone();
   }
 };
 
 /** @type {remoting.HostController} */
 remoting.hostController = null;