Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(86)

Unified Diff: remoting/webapp/unittests/menu_button_unittest.js

Issue 959963002: [Chromoting] Enable jscompile for webapp unittests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/webapp/unittests/menu_button_unittest.js
diff --git a/remoting/webapp/unittests/menu_button_unittest.js b/remoting/webapp/unittests/menu_button_unittest.js
index 7337134d5ac6378487d26aa584fa060db90c28cf..536bb30768163d45fd0813e939bbb77c3db5d24f 100644
--- a/remoting/webapp/unittests/menu_button_unittest.js
+++ b/remoting/webapp/unittests/menu_button_unittest.js
@@ -6,8 +6,11 @@
'use strict';
+/** @type {(sinon.$spy|function():void)} */
var onShow = null;
+/** @type {(sinon.$spy|function():void)} */
var onHide = null;
+/** @type {remoting.MenuButton} */
var menuButton = null;
module('MenuButton', {
@@ -20,11 +23,12 @@ module('MenuButton', {
'<li id="menu-option-1">Option 1</li>' +
'</ul>' +
'</span>';
- onShow = sinon.spy();
- onHide = sinon.spy();
+ onShow = /** @type {(sinon.$spy|function():void)} */ (sinon.spy());
+ onHide = /** @type {(sinon.$spy|function():void)} */ (sinon.spy());
menuButton = new remoting.MenuButton(
document.getElementById('menu-button-container'),
- onShow, onHide);
+ /** @type {function():void} */ (onShow),
+ /** @type {function():void} */ (onHide));
},
teardown: function() {
onShow = null;
@@ -87,4 +91,4 @@ test('select method should set/unset background image', function() {
ok(style.backgroundImage == 'none');
});
-}());
+}());

Powered by Google App Engine
This is Rietveld 408576698