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

Side by Side Diff: third_party/WebKit/LayoutTests/media/media-captions-no-controls.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 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> 4 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
5 5
6 <script src=media-file.js></script> 6 <script src=media-file.js></script>
7 <!-- TODO(philipj): Convert test to testharness.js. crbug.com/588956
8 (Please avoid writing new tests using video-test.js) -->
7 <script src=video-test.js></script> 9 <script src=video-test.js></script>
8 <script src=media-controls.js></script> 10 <script src=media-controls.js></script>
9 <script> 11 <script>
10 12
11 function canplaythrough() 13 function canplaythrough()
12 { 14 {
13 consoleWrite("<br>** Initial state: no text tracks, controls sho uld not be visible **"); 15 consoleWrite("<br>** Initial state: no text tracks, controls sho uld not be visible **");
14 testExpected("video.textTracks.length", 0); 16 testExpected("video.textTracks.length", 0);
15 testExpected("video.controls", false, '=='); 17 testExpected("video.controls", false, '==');
16 18
(...skipping 23 matching lines...) Expand all
40 } 42 }
41 43
42 </script> 44 </script>
43 </head> 45 </head>
44 <body onload="start()"> 46 <body onload="start()">
45 <video> 47 <video>
46 </video> 48 </video>
47 <p>Tests that adding a text track does not make controls visible.</p> 49 <p>Tests that adding a text track does not make controls visible.</p>
48 </body> 50 </body>
49 </html> 51 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698