Add type annotations to media_controls.js.

Add type annotations to media_controls.js.

BUG=444485
TEST=GYP_GENERATORS=ninja tools/gyp/gyp --depth . ui/file_manager/video_player/js/compiled_resources.gyp && ninja -C out/Default | grep "media_controls.js"

Committed: https://crrev.com/45b09e8812165dac19fae75419875b32e2b3cb81
Cr-Commit-Position: refs/heads/master@{#309612}

[yawano, 2014-12-24 01:21:38]

yawano@chromium.org changed reviewers:
+ hirono@chromium.org

[yawano, 2014-12-24 01:21:39]

PTAL. Thank you!

[hirono, 2014-12-24 01:43:22]

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/common/js/util.js (right):

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/common/js/util.js:405: * Update the cache.
Please fix this while you are here: Update -> Updates

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/video_player...
File ui/file_manager/video_player/js/media_controls.js (right):

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/video_player...
ui/file_manager/video_player/js/media_controls.js:11: * @param {HTMLElement}
containerElement The container for the controls.
nit: !HTMLElement

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/video_player...
ui/file_manager/video_player/js/media_controls.js:222: * Toggle play/pause state
on a mouse click on the play/pause button.
nit: Toggle -> Toggles

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/video_player...
ui/file_manager/video_player/js/media_controls.js:742: * Set value formatting
function.
nit: Sets

[yawano, 2014-12-24 02:39:55]

Thank you for the review! PTAL.

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/common/js/util.js (right):

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/common/js/util.js:405: * Update the cache.
On 2014/12/24 01:43:22, hirono wrote:
> Please fix this while you are here: Update -> Updates

Done.

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/video_player...
File ui/file_manager/video_player/js/media_controls.js (right):

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/video_player...
ui/file_manager/video_player/js/media_controls.js:11: * @param {HTMLElement}
containerElement The container for the controls.
On 2014/12/24 01:43:22, hirono wrote:
> nit: !HTMLElement

Done.

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/video_player...
ui/file_manager/video_player/js/media_controls.js:222: * Toggle play/pause state
on a mouse click on the play/pause button.
On 2014/12/24 01:43:22, hirono wrote:
> nit: Toggle -> Toggles

Done.

https://codereview.chromium.org/824973002/diff/1/ui/file_manager/video_player...
ui/file_manager/video_player/js/media_controls.js:742: * Set value formatting
function.
On 2014/12/24 01:43:22, hirono wrote:
> nit: Sets

Done.

[hirono, 2014-12-24 02:42:08]

lgtm!

[yawano, 2014-12-24 03:26:06]

On 2014/12/24 02:42:08, hirono wrote:
> lgtm!

Thank you!

[yawano, 2014-12-24 03:26:11]

The CQ bit was checked by yawano@chromium.org

[commit-bot: I haz the power, 2014-12-24 03:27:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/824973002/20001

[commit-bot: I haz the power, 2014-12-24 04:06:27]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-12-24 04:07:04]

Patchset 2 (id:??) landed as
https://crrev.com/45b09e8812165dac19fae75419875b32e2b3cb81
Cr-Commit-Position: refs/heads/master@{#309612}