Add shortcuts to audio player.

Add shortcuts to audio player.

Space, K - pause
Down/Up or Left/Right - rewind/forward min(5s, 10% of the duration).
J/L - rewind/forward min(10s, 20% of the duration).

BUG=214861
Committed: https://crrev.com/d03ae7a55e5f914eff4683634cc13354ceb1eac1
Cr-Commit-Position: refs/heads/master@{#387257}

[oka, 2016-04-14 05:10:56]

Nit comments.

[oka, 2016-04-14 05:11:27]

PTAL

[oka, 2016-04-14 05:11:39]

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

[oka, 2016-04-14 05:11:39]

PTAL

[fukino, 2016-04-14 05:42:47]

https://codereview.chromium.org/1883813005/diff/20001/ui/file_manager/audio_p...
File ui/file_manager/audio_player/js/audio_player.js (right):

https://codereview.chromium.org/1883813005/diff/20001/ui/file_manager/audio_p...
ui/file_manager/audio_player/js/audio_player.js:376: case 'Right':
Please keep the behaviors on Up/Down.

[oka, 2016-04-14 05:55:22]

Description was changed from

==========
Add shortcuts to audio player.

Space, K - pause
Left/Right - rewind/forward min(5s, 10% of the duration), send to start/end when
paused
J/L - rewind/forward min(10s, 20% of the duration), send to start/end when
paused

BUG=214861
==========

to

==========
Add shortcuts to audio player.

Space, K - pause
Down/Up or Left/Right - rewind/forward min(5s, 10% of the duration), send to
start/end when paused
J/L - rewind/forward min(10s, 20% of the duration), send to start/end when
paused

BUG=214861
==========

[oka, 2016-04-14 05:55:25]

Address comments

[oka, 2016-04-14 05:55:30]

PTAL

https://codereview.chromium.org/1883813005/diff/20001/ui/file_manager/audio_p...
File ui/file_manager/audio_player/js/audio_player.js (right):

https://codereview.chromium.org/1883813005/diff/20001/ui/file_manager/audio_p...
ui/file_manager/audio_player/js/audio_player.js:376: case 'Right':
On 2016/04/14 05:42:47, fukino wrote:
> Please keep the behaviors on Up/Down.

Done.

[oka, 2016-04-14 06:03:50]

Revert 'send to start when paused' for now.

[oka, 2016-04-14 06:04:45]

Description was changed from

==========
Add shortcuts to audio player.

Space, K - pause
Down/Up or Left/Right - rewind/forward min(5s, 10% of the duration), send to
start/end when paused
J/L - rewind/forward min(10s, 20% of the duration), send to start/end when
paused

BUG=214861
==========

to

==========
Add shortcuts to audio player.

Space, K - pause
Down/Up or Left/Right - rewind/forward min(5s, 10% of the duration).
J/L - rewind/forward min(10s, 20% of the duration).

BUG=214861
==========

[oka, 2016-04-14 06:05:02]

PTAL

[fukino, 2016-04-14 06:52:19]

lgtm

[oka, 2016-04-14 07:05:24]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2016-04-14 07:05:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1883813005/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1883813005/60001

[commit-bot: I haz the power, 2016-04-14 07:45:17]

Description was changed from

==========
Add shortcuts to audio player.

Space, K - pause
Down/Up or Left/Right - rewind/forward min(5s, 10% of the duration).
J/L - rewind/forward min(10s, 20% of the duration).

BUG=214861
==========

to

==========
Add shortcuts to audio player.

Space, K - pause
Down/Up or Left/Right - rewind/forward min(5s, 10% of the duration).
J/L - rewind/forward min(10s, 20% of the duration).

BUG=214861
==========

[commit-bot: I haz the power, 2016-04-14 07:45:17]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-04-14 07:47:36]

Description was changed from

==========
Add shortcuts to audio player.

Space, K - pause
Down/Up or Left/Right - rewind/forward min(5s, 10% of the duration).
J/L - rewind/forward min(10s, 20% of the duration).

BUG=214861
==========

to

==========
Add shortcuts to audio player.

Space, K - pause
Down/Up or Left/Right - rewind/forward min(5s, 10% of the duration).
J/L - rewind/forward min(10s, 20% of the duration).

BUG=214861

Committed: https://crrev.com/d03ae7a55e5f914eff4683634cc13354ceb1eac1
Cr-Commit-Position: refs/heads/master@{#387257}
==========

[commit-bot: I haz the power, 2016-04-14 07:47:37]

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