[Chromoting] Enable jscompile for browser tests.

remoting/webapp/unittests/chrome_mocks.js

 // Copyright 2014 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.
 
 // This file contains various mock objects for the chrome platform to make
 // unit testing easier.
 
 Entry = function() {};
 
-(function(scope){
+(function(){
 
 var chromeMocks = {};
 
+/**
+ * @constructor
+ */
 chromeMocks.Event = function() {
   this.listeners_ = [];
 };
 
+/** @param {Function} callback */
 chromeMocks.Event.prototype.addListener = function(callback) {
   this.listeners_.push(callback);
 };
 
+/** @param {Function} callback */
 chromeMocks.Event.prototype.removeListener = function(callback) {
   for (var i = 0; i < this.listeners_.length; i++) {
     if (this.listeners_[i] === callback) {
       this.listeners_.splice(i, 1);
       break;
     }
   }
 };
 
+/**
+ * @param {...*} var_args
+ * @return {void}
+ */
 chromeMocks.Event.prototype.mock$fire = function(var_args) {
   var params = Array.prototype.slice.call(arguments);
-  this.listeners_.forEach(function(listener){
-    listener.apply(null, params);
-  });
+  this.listeners_.forEach(
+      /** @param {Function} listener */
+      function(listener){
+        listener.apply(null, params);
+      });
 };
 
+/** @type {Object} */
 chromeMocks.runtime = {};
 
+/** @constructor */
 chromeMocks.runtime.Port = function() {
   this.onMessage = new chromeMocks.Event();
   this.onDisconnect = new chromeMocks.Event();
 
+  /** @type {string} */
   this.name = '';
+
+  /** @type {chrome.runtime.MessageSender} */
   this.sender = null;
 };
 
 chromeMocks.runtime.Port.prototype.disconnect = function() {};
-chromeMocks.runtime.Port.prototype.postMessage = function() {};
 
+/**
+ * @param {Object} message
+ */
+chromeMocks.runtime.Port.prototype.postMessage = function(message) {};
+
+/** @type {chromeMocks.Event} */
 chromeMocks.runtime.onMessage = new chromeMocks.Event();
-chromeMocks.runtime.sendMessage = function(extensionId, message,
+
+/**
+ * @param {string?} extensionId
+ * @param {*} message
+ * @param {Object=} options
+ * @param {function(*)=} responseCallback
+ */
+chromeMocks.runtime.sendMessage = function(extensionId, message, options,
                                            responseCallback) {
   base.debug.assert(
       extensionId === null,
       'The mock only supports sending messages to the same extension.');
   extensionId = chrome.runtime.id;
   window.requestAnimationFrame(function() {
     var message_copy = base.deepCopy(message);
     chromeMocks.runtime.onMessage.mock$fire(
         message_copy, {id: extensionId}, responseCallback);
   });
 };
 
+/** @type {string} */
 chromeMocks.runtime.id = 'extensionId';
 
+/** @type {Object} */
 chromeMocks.storage = {};
 
 // Sample implementation of chrome.StorageArea according to
 // https://developer.chrome.com/apps/storage#type-StorageArea
+/** @constructor */
 chromeMocks.StorageArea = function() {
+  /** @type {Object} */
   this.storage_ = {};
 };
 
+/**
+ * @param {!Object} keys
+ * @return {Array.<string>}

[kelvinp, 2015/02/18 18:41:30]

dbeam recently switch all Array.<string> into Array<string>
https://codereview.chromium.org/917093003

[garykac, 2015/02/19 01:48:45]

Done.

+ */
 function getKeys(keys) {
   if (typeof keys === 'string') {
     return [keys];
   } else if (typeof keys === 'object') {
     return Object.keys(keys);
   }
   return [];
 }
 
+/**
+ * @param {!Object} keys
+ * @param {Function} onDone
+ */
 chromeMocks.StorageArea.prototype.get = function(keys, onDone) {
   if (!keys) {
     onDone(base.deepCopy(this.storage_));
     return;
   }
 
   var result = (typeof keys === 'object') ? keys : {};
-  getKeys(keys).forEach(function(key) {
-    if (key in this.storage_) {
-      result[key] = base.deepCopy(this.storage_[key]);
-    }
-  }, this);
+  getKeys(keys).forEach(
+      /** @param {string} key */
+      function(key) {
+        if (key in this.storage_) {
+          result[key] = base.deepCopy(this.storage_[key]);
+        }
+      }, this);
   onDone(result);
 };
 
+/** @param {Object} value */
 chromeMocks.StorageArea.prototype.set = function(value) {
   for (var key in value) {
     this.storage_[key] = base.deepCopy(value[key]);
   }
 };
 
+/**
+ * @param {!Object} keys
+ */
 chromeMocks.StorageArea.prototype.remove = function(keys) {
-  getKeys(keys).forEach(function(key) {
-    delete this.storage_[key];
-  }, this);
+  getKeys(keys).forEach(
+      /** @param {string} key */
+      function(key) {
+        delete this.storage_[key];
+      }, this);
 };
 
 chromeMocks.StorageArea.prototype.clear = function() {
   this.storage_ = null;
 };
 
+/** @type {chromeMocks.StorageArea} */
 chromeMocks.storage.local = new chromeMocks.StorageArea();
 
 var originals_ = null;
 
 /**
  * Activates a list of Chrome components to mock
  * @param {Array<string>} components
  */
 chromeMocks.activate = function(components) {
   if (originals_) {
@@ skipping 12 matching lines @@
 chromeMocks.restore = function() {
   if (!originals_) {
     throw new Error('You must call activate() before restore().');
   }
   for (var components in originals_) {
     chrome[components] = originals_[components];
   }
   originals_ = null;
 };
 
-scope.chromeMocks = chromeMocks;
-
-})(window);
+})();