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

Side by Side Diff: third_party/WebKit/LayoutTests/media/track/regions-webvtt/text-track-cue-region-attribute.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> 10 <script>
9 var cue; 11 var cue;
10 12
11 function startTest() 13 function startTest()
12 { 14 {
13 if (!window.VTTRegion) { 15 if (!window.VTTRegion) {
14 failTest(); 16 failTest();
15 return; 17 return;
16 } 18 }
(...skipping 22 matching lines...) Expand all
39 } 41 }
40 </script> 42 </script>
41 </head> 43 </head>
42 <body> 44 <body>
43 <p>Tests the regionId attribute of a cue.</p> 45 <p>Tests the regionId attribute of a cue.</p>
44 <video controls> 46 <video controls>
45 <track src="../captions-webvtt/header-regions.vtt" kind="captions" d efault onload="startTest()"> 47 <track src="../captions-webvtt/header-regions.vtt" kind="captions" d efault onload="startTest()">
46 </video> 48 </video>
47 </body> 49 </body>
48 </html> 50 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698