remoting: Fix JSCompiler issues blocking ES6 transition.

remoting/webapp/unittests/spy_promise_unittest.js

 // Copyright 2015 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';
 
 var originalGlobalPromise = Promise;
 
@@ skipping 12 matching lines @@
   assert.ok(
       !base.SpyPromise.isSettleAllRunning(),
       'settleAll should not be running');
   assert.equal(
       base.SpyPromise.unsettledCount, 0,
       'base.SpyPromise.unsettledCount should be zero ' +
         'before/after any test finishes');
 }
 
 /**
+ * @param {!QUnit.Assert} assert
  * @return {!Promise}
  */
-function finish(/** QUnit.Assert */ assert) {
+function finish(assert) {
   return base.SpyPromise.settleAll().then(function() {
     assert.equal(
         base.SpyPromise.unsettledCount, 0,
         'base.SpyPromise.unsettledCount should be zero ' +
           'after settleAll finishes.');
   });
 }
 
 QUnit.test('run', function(assert) {
   var done = assert.async();
@@ skipping 90 matching lines @@
     assert.equal(value, 'hello');
     done();
   });
   assert.equal(base.SpyPromise.unsettledCount, 1);
   return finish(assert);
 });
 
 QUnit.test('all', function(assert) {
   var done = assert.async();
   base.SpyPromise.all([Promise.resolve(1), Promise.resolve(2)]).
-      then(function(/**string*/ value) {
-        assert.equal(base.SpyPromise.unsettledCount, 0);
-        assert.deepEqual(value, [1, 2]);
-        done();
-      });
+      then(
+          /** @param {string} value */
+          function(value) {
+            assert.equal(base.SpyPromise.unsettledCount, 0);
+            assert.deepEqual(value, [1, 2]);
+            done();
+          });
   assert.equal(base.SpyPromise.unsettledCount, 1);
   return finish(assert);
 });
 
 QUnit.test('race', function(assert) {
   var done = assert.async();
   var fast = Promise.resolve('fast');
   var slow = new Promise(function() {}); // never settled
   base.SpyPromise.race([fast, slow]).
       then(function(/**string*/ value) {
@@ skipping 80 matching lines @@
   var done = assert.async();
   var startTime = Date.now();
   var neverResolved = new base.SpyPromise(function() {});
   return base.SpyPromise.settleAll(1000).catch(function(error) {
     assert.ok(startTime + 1500 < Date.now());
     done();
   });
 });
 
 })();