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

Side by Side Diff: third_party/WebKit/LayoutTests/media/video-controls-captions-multiple-clicks.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 <title>Test closed caption button toggling.</title> 5 <title>Test closed caption button toggling.</title>
6 <script src=media-file.js></script> 6 <script src=media-file.js></script>
7 <script src=media-controls.js></script> 7 <script src=media-controls.js></script>
8 <!-- TODO(philipj): Convert test to testharness.js. crbug.com/588956
9 (Please avoid writing new tests using video-test.js) -->
8 <script src=video-test.js></script> 10 <script src=video-test.js></script>
9 <script> 11 <script>
10 var displayElement; 12 var displayElement;
11 var track; 13 var track;
12 var text = ["First", "Second", "Third"]; 14 var text = ["First", "Second", "Third"];
13 15
14 function addTextTrack() 16 function addTextTrack()
15 { 17 {
16 track = video.addTextTrack('captions'); 18 track = video.addTextTrack('captions');
17 19
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 79
78 video.src = findMediaFile('video', 'content/counting'); 80 video.src = findMediaFile('video', 'content/counting');
79 } 81 }
80 </script> 82 </script>
81 </head> 83 </head>
82 <body onload="loaded()"> 84 <body onload="loaded()">
83 <p>Tests that multiple toggles of the closed captions button still display c aptions</p> 85 <p>Tests that multiple toggles of the closed captions button still display c aptions</p>
84 <video controls></video> 86 <video controls></video>
85 </body> 87 </body>
86 </html> 88 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698