Chromium Code Reviews| Index: ui/file_manager/video_player/js/video_player.js |
| diff --git a/ui/file_manager/video_player/js/video_player.js b/ui/file_manager/video_player/js/video_player.js |
| index 20c101c75827a920589e2756d21156bde281c62f..206deaeaee5d1af6c4b978fefe6a477fb535f9e3 100644 |
| --- a/ui/file_manager/video_player/js/video_player.js |
| +++ b/ui/file_manager/video_player/js/video_player.js |
| @@ -25,16 +25,29 @@ function FullWindowVideoControls( |
| this.updateStyle(); |
| window.addEventListener('resize', this.updateStyle.wrap(this)); |
| document.addEventListener('keydown', function(e) { |
| - if (e.keyIdentifier == 'U+0020') { // Space |
| - this.togglePlayStateWithFeedback(); |
| - e.preventDefault(); |
| - } |
| - if (e.keyIdentifier == 'U+001B') { // Escape |
| - util.toggleFullScreen( |
| - chrome.app.window.current(), |
| - false); // Leave the full screen mode. |
| - e.preventDefault(); |
| + switch (e.keyIdentifier) { |
| + case 'U+0020': // Space |
| + case 'MediaPlayPause': |
| + this.togglePlayStateWithFeedback(); |
| + break; |
| + case 'U+001B': // Escape |
| + util.toggleFullScreen( |
| + chrome.app.window.current(), |
| + false); // Leave the full screen mode. |
| + break; |
| + case 'Right': |
| + case 'MediaNextTrack': |
| + player.advance_(1); |
| + break; |
| + case 'Left': |
| + case 'MediaPreviousTrack': |
| + player.advance_(0); |
| + break; |
| + case 'MediaStop': |
| + //TODO: Define "Stop" behavior. |
|
yoshiki
2014/07/14 23:48:39
nit: add a space before 'TODO'.
etrunko
2014/07/15 13:31:14
Done.
|
| + break; |
| } |
| + e.preventDefault(); |
| }.wrap(this)); |
| // TODO(mtomasz): Simplify. crbug.com/254318. |