Fix jscompile errors.

remoting/webapp/remoting.js

Index: remoting/webapp/remoting.js
diff --git a/remoting/webapp/remoting.js b/remoting/webapp/remoting.js
index 7bc5931f39cd84a2eef796cf3922078992e1646f..188dc062da3a3c2b1557a176f82a59353a8bb859 100644
--- a/remoting/webapp/remoting.js
+++ b/remoting/webapp/remoting.js
@@ -529,7 +529,7 @@ remoting.generateXsrfToken = function() {
 /**
  * Tests whether we are running on Mac.
  *
- * @return {bool} True if the platform is Mac.
+ * @return {boolean} True if the platform is Mac.
  */
 remoting.platformIsMac = function() {
   return navigator.platform.indexOf('Mac') != -1;
@@ -538,7 +538,7 @@ remoting.platformIsMac = function() {
 /**
  * Tests whether we are running on Windows.
  *
- * @return {bool} True if the platform is Windows.
+ * @return {boolean} True if the platform is Windows.
  */
 remoting.platformIsWindows = function() {
   return navigator.platform.indexOf('Win32') != -1;
@@ -547,7 +547,7 @@ remoting.platformIsWindows = function() {
 /**
  * Tests whether we are running on Linux.
  *
- * @return {bool} True if the platform is Linux.
+ * @return {boolean} True if the platform is Linux.
  */
 remoting.platformIsLinux = function() {
   return (navigator.platform.indexOf('Linux') != -1) &&
@@ -557,8 +557,8 @@ remoting.platformIsLinux = function() {
 /**
  * Tests whether we are running on ChromeOS.
  *
- * @return {bool} True if the platform is ChromeOS.
+ * @return {boolean} True if the platform is ChromeOS.
  */
 remoting.platformIsChromeOS = function() {
-  return navigator.userAgent.match(/\bCrOS\b/);
+  return navigator.userAgent.match(/\bCrOS\b/) != null;

[Wez, 2014/08/05 00:42:27]

Is this the preferred pattern, versus !!foo?

[Jamie, 2014/08/05 01:03:10]

I don't think there's a consensus. I think I prefer this for clarity.

 }