Video Player: Don't show the loop message in unexpected timing

Video Player: Don't show the loop message in unexpected timing

BUG=418275
TEST=manually tested

Committed: https://crrev.com/b885fc3b198e7d8772d1e8c4e4f93d470eb65ae1
Cr-Commit-Position: refs/heads/master@{#297151}

[yoshiki, 2014-09-29 07:56:58]

yoshiki@chromium.org changed reviewers:
+ fukino@chromium.org

[yoshiki, 2014-09-29 07:57:09]

@fukino-san, PTAL. Thanks.

[fukino, 2014-09-29 08:04:29]

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

https://codereview.chromium.org/611953002/diff/1/ui/file_manager/video_player...
ui/file_manager/video_player/js/media_controls.js:1035: var onAnimationEnd =
function(text, event) {
This 'text' looks unused. Can we get rid of it?

[yoshiki, 2014-09-29 08:10:19]

@fukino, PTAL

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

https://codereview.chromium.org/611953002/diff/1/ui/file_manager/video_player...
ui/file_manager/video_player/js/media_controls.js:1035: var onAnimationEnd =
function(text, event) {
On 2014/09/29 08:04:29, fukino wrote:
> This 'text' looks unused. Can we get rid of it?

Done.

[fukino, 2014-09-29 08:10:59]

On 2014/09/29 08:10:19, yoshiki wrote:
> @fukino, PTAL
> 
>
https://codereview.chromium.org/611953002/diff/1/ui/file_manager/video_player...
> File ui/file_manager/video_player/js/media_controls.js (right):
> 
>
https://codereview.chromium.org/611953002/diff/1/ui/file_manager/video_player...
> ui/file_manager/video_player/js/media_controls.js:1035: var onAnimationEnd =
> function(text, event) {
> On 2014/09/29 08:04:29, fukino wrote:
> > This 'text' looks unused. Can we get rid of it?
> 
> Done.

lgtm.

[yoshiki, 2014-09-29 08:11:25]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2014-09-29 08:12:22]

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

[commit-bot: I haz the power, 2014-09-29 08:50:21]

Committed patchset #2 (id:20001) as eeee6758718c07a716d622dcbe55959c04be1cb1

[commit-bot: I haz the power, 2014-09-29 08:51:06]

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