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

Side by Side Diff: third_party/WebKit/LayoutTests/media/video-controls-attribute-fullscreen.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 <head> 3 <head>
4 <title>Test that the controls attribute is not affected by fullscreen</t itle> 4 <title>Test that the controls attribute is not affected by fullscreen</t itle>
5 <script src="media-file.js"></script> 5 <script src="media-file.js"></script>
6 <!-- TODO(philipj): Convert test to testharness.js. crbug.com/588956
7 (Please avoid writing new tests using video-test.js) -->
6 <script src="video-test.js"></script> 8 <script src="video-test.js"></script>
7 <script> 9 <script>
8 function start() 10 function start()
9 { 11 {
10 findMediaElement(); 12 findMediaElement();
11 video.src = findMediaFile("video", "content/test"); 13 video.src = findMediaFile("video", "content/test");
12 waitForEvent("loadeddata", loadeddata); 14 waitForEvent("loadeddata", loadeddata);
13 } 15 }
14 16
15 function loadeddata() 17 function loadeddata()
(...skipping 13 matching lines...) Expand all
29 testExpected("video.controls", false); 31 testExpected("video.controls", false);
30 endTest(); 32 endTest();
31 } 33 }
32 </script> 34 </script>
33 </head> 35 </head>
34 <body onload="start()"> 36 <body onload="start()">
35 <p>Test that the controls attribute is not affected by fullscreen</p> 37 <p>Test that the controls attribute is not affected by fullscreen</p>
36 <video></video> 38 <video></video>
37 </body> 39 </body>
38 </html> 40 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698