[Chromoting] Enable jscompile for webapp unittests.

remoting/webapp/unittests/event_hook_unittest.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.
 
 (function() {
 
 'use strict';
 
+/** @type {base.EventSourceImpl} */
 var eventSource = null;
+
+/** @type {HTMLElement} */
 var domElement = null;
+
+/** @type {chrome.Event} */
 var myChromeEvent = null;
+
+/** @type {Listener} */
 var listener = null;
 
+/** @constructor */
 var Listener = function(element) {
-  this.onChromeEvent = sinon.stub();
-  this.onClickEvent = sinon.stub();
-  this.onCustomEvent = sinon.stub();
+  /** @type {(sinon.$spy|function(...?))} */
+  this.onChromeEvent = sinon.spy();
+  /** @type {(sinon.$spy|function(...?))} */
+  this.onClickEvent = sinon.spy();
+  /** @type {(sinon.$spy|function(...?))} */
+  this.onCustomEvent = sinon.spy();
+
   this.eventHooks_ = new base.Disposables(
-      new base.DomEventHook(element, 'click', this.onClickEvent.bind(this),
+      new base.DomEventHook(domElement, 'click', this.onClickEvent.bind(this),

[kelvinp, 2015/02/26 00:31:30]

should be element as it is passed through the constructor.  Or you can keep it
to be domElement and remove element from the constructor

[garykac, 2015/02/28 02:33:33]

I didn't see the 'element' param and thought it was undefined.

Reverted and added @param for element.

                             false),
       new base.EventHook(eventSource, 'customEvent',
                          this.onCustomEvent.bind(this)),
       new base.ChromeEventHook(myChromeEvent, this.onChromeEvent.bind(this)));
 };
 
 Listener.prototype.dispose = function() {
   this.eventHooks_.dispose();
 };
 
 function raiseAllEvents() {
   domElement.click();
   myChromeEvent.mock$fire();
   eventSource.raiseEvent('customEvent');
 }
 
 module('base.EventHook', {
   setup: function() {
-    domElement = document.createElement('div');
+    domElement = /** @type {HTMLElement} */ (document.createElement('div'));
     eventSource = new base.EventSourceImpl();
     eventSource.defineEvents(['customEvent']);
-    myChromeEvent = new chromeMocks.Event();
+    myChromeEvent = new chrome.Event();
     listener = new Listener(domElement);
   },
   tearDown: function() {
     domElement = null;
     eventSource = null;
     myChromeEvent = null;
     listener = null;
   }
 });
 
 test('EventHook should hook events when constructed', function() {
   raiseAllEvents();
   sinon.assert.calledOnce(listener.onClickEvent);
   sinon.assert.calledOnce(listener.onChromeEvent);
   sinon.assert.calledOnce(listener.onCustomEvent);
   listener.dispose();
 });
 
 test('EventHook should unhook events when disposed', function() {
   listener.dispose();
   raiseAllEvents();
   sinon.assert.notCalled(listener.onClickEvent);
   sinon.assert.notCalled(listener.onChromeEvent);
   sinon.assert.notCalled(listener.onCustomEvent);
 });
 
 })();