Let a video resume automatically when changing the playback position after the playback has finished

Let a video resume automatically when changing the playback position after the playback has finished

BUG=418441
TEST=manually tested

Committed: https://crrev.com/4b974a1a40c855946a0a9ece10aa9647e31d97b1
Cr-Commit-Position: refs/heads/master@{#413424}

[harukam1, 2016-08-22 06:39:02]

harukam@chromium.org changed reviewers:
+ fukino@chromium.org, harukam@chromium.org

[harukam1, 2016-08-22 06:39:03]

PTAL

[fukino, 2016-08-22 07:22:01]

LGTM with a nit.

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

https://codereview.chromium.org/2263033002/diff/1/ui/file_manager/video_playe...
ui/file_manager/video_player/js/media_controls.js:348: if (this.media_.ended)
nit: Please add a comment explaining that we should re-start playing the video
when we drag the seek bar from ending point.

[harukam1, 2016-08-22 08:34:03]

PTAL

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

https://codereview.chromium.org/2263033002/diff/1/ui/file_manager/video_playe...
ui/file_manager/video_player/js/media_controls.js:348: if (this.media_.ended)
On 2016/08/22 07:22:01, fukino wrote:
> nit: Please add a comment explaining that we should re-start playing the video
> when we drag the seek bar from ending point.

Acknowledged.

[fukino, 2016-08-22 08:41:59]

fukino@chromium.org changed reviewers:
- harukam@chromium.org

[fukino, 2016-08-22 08:41:59]

still lgtm. Thanks!

[fukino, 2016-08-22 08:42:02]

still lgtm. Thanks!

[harukam1, 2016-08-22 08:47:06]

The CQ bit was checked by harukam@chromium.org

[commit-bot: I haz the power, 2016-08-22 08:47:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-22 09:31:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-22 09:34:44]

Description was changed from

==========
Let a video resume automatically when changing the playback position after the
playback has finished

BUG=418441
TEST=manually tested
==========

to

==========
Let a video resume automatically when changing the playback position after the
playback has finished

BUG=418441
TEST=manually tested

Committed: https://crrev.com/4b974a1a40c855946a0a9ece10aa9647e31d97b1
Cr-Commit-Position: refs/heads/master@{#413424}
==========

[commit-bot: I haz the power, 2016-08-22 09:34:44]

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