Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(20)

Side by Side Diff: third_party/WebKit/LayoutTests/media/controls/controls-video-keynav.html

Issue 2938913002: Updates volume to change by .05 increments. (Closed)
Patch Set: Addresses comment 22. Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | third_party/WebKit/Source/modules/media_controls/MediaControlsImpl.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <DOCTYPE html> 1 <DOCTYPE html>
2 <title>Test media controls video keyboard navigation</title> 2 <title>Test media controls video keyboard navigation</title>
3 <script src="../../resources/testharness.js"></script> 3 <script src="../../resources/testharness.js"></script>
4 <script src="../../resources/testharnessreport.js"></script> 4 <script src="../../resources/testharnessreport.js"></script>
5 <script src="../media-file.js"></script> 5 <script src="../media-file.js"></script>
6 <script src="../media-controls.js"></script> 6 <script src="../media-controls.js"></script>
7 <video controls></video> 7 <video controls></video>
8 <script> 8 <script>
9 test(_ => { 9 test(_ => {
10 assert_true('eventSender' in window); 10 assert_true('eventSender' in window);
11 }, 'eventSender required'); 11 }, 'eventSender required');
12 12
13 async_test(function(t) { 13 async_test(function(t) {
14 var video = document.querySelector("video"); 14 var video = document.querySelector("video");
15 video.src = findMediaFile("video", "../content/test"); 15 video.src = findMediaFile("video", "../content/test");
16 assert_equals(video.volume, 1); 16 assert_equals(video.volume, 1);
17 assert_equals(video.currentTime, 0); 17 assert_equals(video.currentTime, 0);
18 18
19 video.oncanplaythrough = t.step_func_done(function() { 19 video.oncanplaythrough = t.step_func_done(function() {
20 // Focus the video. 20 // Focus the video.
21 video.focus(); 21 video.focus();
22 22
23 // 'Enter' plays the video. 23 // 'Enter' plays the video.
24 eventSender.keyDown("Enter"); 24 eventSender.keyDown("Enter");
25 assert_false(video.paused); 25 assert_false(video.paused);
26 26
27 // 'Down' reduces volume. 27 // 'Down' reduces volume by .05.
28 eventSender.keyDown("ArrowDown"); 28 eventSender.keyDown("ArrowDown");
29 assert_less_than(video.volume, 1); 29 assert_equals(video.volume, .95);
30 30
31 // 'Up' increases volume. 31 eventSender.keyDown("ArrowDown");
32 assert_equals(video.volume, .9);
33
34 // 'Up' increases volume by .05
35 eventSender.keyDown("ArrowUp");
36 assert_equals(video.volume, .95);
37
32 eventSender.keyDown("ArrowUp"); 38 eventSender.keyDown("ArrowUp");
33 assert_equals(video.volume, 1); 39 assert_equals(video.volume, 1);
34 40
35 // 'Enter' again pauses the video. 41 // 'Enter' again pauses the video.
36 eventSender.keyDown("Enter"); 42 eventSender.keyDown("Enter");
37 assert_true(video.paused); 43 assert_true(video.paused);
38 44
39 // 'Space' also pauses/plays the video. 45 // 'Space' also pauses/plays the video.
40 eventSender.keyDown(" "); 46 eventSender.keyDown(" ");
41 assert_false(video.paused); 47 assert_false(video.paused);
(...skipping 16 matching lines...) Expand all
58 // 'End' sets the timeline to end. 64 // 'End' sets the timeline to end.
59 eventSender.keyDown("End"); 65 eventSender.keyDown("End");
60 assert_equals(video.currentTime, video.duration); 66 assert_equals(video.currentTime, video.duration);
61 67
62 // 'Left' scrubs the timeline back. 68 // 'Left' scrubs the timeline back.
63 eventSender.keyDown("ArrowLeft"); 69 eventSender.keyDown("ArrowLeft");
64 assert_less_than(video.currentTime, video.duration); 70 assert_less_than(video.currentTime, video.duration);
65 }); 71 });
66 }); 72 });
67 </script> 73 </script>
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/Source/modules/media_controls/MediaControlsImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698