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

remoting/webapp/crd/js/local_host_section.js

Index: remoting/webapp/crd/js/local_host_section.js
diff --git a/remoting/webapp/crd/js/local_host_section.js b/remoting/webapp/crd/js/local_host_section.js
index 50bebe7cf0420732084034244565d0af5614b3b6..ac676818e1ad86de0d1fb5c0d6777b362fae821f 100644
--- a/remoting/webapp/crd/js/local_host_section.js
+++ b/remoting/webapp/crd/js/local_host_section.js
@@ -107,6 +107,23 @@ remoting.LocalHostSection.prototype.canChangeState = function() {
   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_);