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

Side by Side Diff: third_party/WebKit/LayoutTests/media/video-controls-focus-movement-on-hide.html

Issue 1715303002: Add TODOs to convert from video-test.js to testharness.js (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add bug ref Created 4 years, 10 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
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <html> 2 <html>
3 <title>Test focus movement when controls fade out with a button focused</title> 3 <title>Test focus movement when controls fade out with a button focused</title>
4 <!-- TODO(philipj): Convert test to testharness.js. crbug.com/588956
5 (Please avoid writing new tests using video-test.js) -->
4 <script src=video-test.js></script> 6 <script src=video-test.js></script>
5 <script src=media-controls.js></script> 7 <script src=media-controls.js></script>
6 <p> 8 <p>
7 Test that focus moves to the &lt;body>/document when controls fade out with a bu tton focused. 9 Test that focus moves to the &lt;body>/document when controls fade out with a bu tton focused.
8 </p> 10 </p>
9 <video controls src="content/test.ogv"></video> 11 <video controls src="content/test.ogv"></video>
10 <script> 12 <script>
11 var controls; 13 var controls;
12 var video = document.querySelector("video"); 14 var video = document.querySelector("video");
13 15
(...skipping 13 matching lines...) Expand all
27 testExpected("getComputedStyle(controls).opacity", 0); 29 testExpected("getComputedStyle(controls).opacity", 0);
28 30
29 testExpected("document.activeElement", document.body); 31 testExpected("document.activeElement", document.body);
30 32
31 consoleWrite(""); 33 consoleWrite("");
32 endTest(); 34 endTest();
33 }, video); 35 }, video);
34 }); 36 });
35 37
36 </script> 38 </script>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698