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

remoting/webapp/crd/js/local_host_section.js

 // Copyright 2015 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.
 
 var remoting = remoting || {};
 
 (function() {
 
 'use strict';
 
@@ skipping 89 matching lines @@
       state === remoting.HostController.State.NOT_INSTALLED) {
     return false;
   }
 
   // In addition, it cannot be started if there is an error (in many error
   // states, the start operation will fail anyway, but even if it succeeds, the
   // chance of a related but hard-to-diagnose future error is high).
   return this.isEnabled_() || !this.hasError_;
 };
 
+/**
+ * Returns true if the current platform is fully supported. It's only used when
+ * we detect that host native messaging components are not installed. In that
+ * case the result of this function determines if the webapp should show the
+ * controls that allow to install and enable Me2Me host.
+ *
+ * @return {boolean}
+ */
+remoting.isMe2MeInstallable = function() {

[garykac, 2015/03/19 01:03:44]

moved here from remoting.js

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

It looks like the function is only used here.
Please make it either an anonymous function or a private member of
LocalHostSecion.

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

Done.

+  // The chromoting host is currently not installable on ChromeOS.
+  // For Linux, we have a install package for Ubuntu but not other distros.
+  // Since we cannot tell from javascript alone the Linux distro the client is
+  // on, we don't show the daemon-control UI for Linux unless the host is
+  // installed.
+  return remoting.platformIsWindows() || remoting.platformIsMac();
+}
+
 /** @private */
 remoting.LocalHostSection.prototype.updateUI_ = function() {
   this.hostTableEntry_.setHost(this.host_);
 
   // Disable elements.
   var enabled = this.isEnabled_();
   var canChangeLocalHostState = this.canChangeState();
   var daemonState = '';
   if (!enabled) {
     daemonState = 'disabled';
@@ skipping 33 matching lines @@
   this.setupDialog_.showForPin();
 };
 
 /** @param {remoting.HostTableEntry} host */
 remoting.LocalHostSection.Controller.prototype.rename = function(host) {
   this.hostList_.renameHost(host);
 };
 
 }());