Fix jscompile errors.

remoting/webapp/remoting.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 || {};
 
 /** @type {remoting.HostSession} */ remoting.hostSession = null;
@@ skipping 511 matching lines @@
 remoting.generateXsrfToken = function() {
   var random = new Uint8Array(16);
   window.crypto.getRandomValues(random);
   var base64Token = window.btoa(String.fromCharCode.apply(null, random));
   return base64Token.replace(/\+/g, '-').replace(/\//g, '_').replace(/=/g, '');
 };
 
 /**
  * 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;
 }
 
 /**
  * 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;
 }
 
 /**
  * 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) &&
          !remoting.platformIsChromeOS();
 }
 
 /**
  * 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.

 }