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

Side by Side Diff: third_party/WebKit/LayoutTests/media/video-controls-always-visible-when-control-hovered.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 <style> 3 <style>
4 #no-video-media { 4 #no-video-media {
5 width: 320px; 5 width: 320px;
6 height: 240px; 6 height: 240px;
7 } 7 }
8 </style> 8 </style>
9 <!-- TODO(philipj): Convert test to testharness.js. crbug.com/588956
10 (Please avoid writing new tests using video-test.js) -->
9 <script src=video-test.js></script> 11 <script src=video-test.js></script>
10 <script src=media-file.js></script> 12 <script src=media-file.js></script>
11 <script src=media-controls.js></script> 13 <script src=media-controls.js></script>
12 <script> 14 <script>
13 var controls; 15 var controls;
14 16
15 function runTest() 17 function runTest()
16 { 18 {
17 video = document.getElementById("no-video-media"); 19 video = document.getElementById("no-video-media");
18 20
(...skipping 25 matching lines...) Expand all
44 the mouse does not move, so the control is still hovered and it should 46 the mouse does not move, so the control is still hovered and it should
45 remain visible.</p> 47 remain visible.</p>
46 <p>This test only runs in DRT!</p> 48 <p>This test only runs in DRT!</p>
47 49
48 <video id="no-video-media" controls loop oncanplaythrough="runTest()"></vide o> 50 <video id="no-video-media" controls loop oncanplaythrough="runTest()"></vide o>
49 <script> 51 <script>
50 setSrcById("no-video-media", findMediaFile("video", "content/test")); 52 setSrcById("no-video-media", findMediaFile("video", "content/test"));
51 </script> 53 </script>
52 </body> 54 </body>
53 </html> 55 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698