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

Side by Side Diff: LayoutTests/media/track/track-webvtt-tc003-newlines.html

Issue 65493006: Remove a WebVTT newlines test that doesn't do what it says (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: rebased Created 7 years, 1 month 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 <script src=../video-test.js></script> 7 <script src=../video-test.js></script>
8 <script> 8 <script>
9
10 numberOfTrackTests = 2;
11 9
12 function trackLoaded() 10 function trackLoaded()
13 { 11 {
14 numberOfTracksLoaded++;
15 if (numberOfTracksLoaded == numberOfTrackTests) {
16 testTrack(0);
17 testTrack(1);
18 }
19 }
20
21 function testTrack(i)
22 {
23 findMediaElement(); 12 findMediaElement();
24 var expected = 13 var expected =
25 { 14 {
26 length : 1, 15 length : 1,
27 tests: 16 tests:
28 [ 17 [
29 { 18 {
30 property : "id", 19 property : "id",
31 values : [1], 20 values : [1],
32 }, 21 },
33 { 22 {
34 property : "startTime", 23 property : "startTime",
35 values : [0.0], 24 values : [0.0],
36 }, 25 },
37 { 26 {
38 property : "endTime", 27 property : "endTime",
39 values : [30.5], 28 values : [30.5],
40 }, 29 },
41 { 30 {
42 property : "text", 31 property : "text",
43 values : ["Bear is Coming!!!!!"], 32 values : ["Bear is Coming!!!!!"],
44 }, 33 },
45 ], 34 ],
46 }; 35 };
47 testCues(i, expected); 36 testCues(0, expected);
48 37
49 allTestsEnded(); 38 endTest();
50 } 39 }
51 </script> 40 </script>
52 </head> 41 </head>
53 <body onload="enableAllTextTracks()"> 42 <body>
54 <p>Tests that line terminators \r, \n, or \r\n are properly parsed, even when there is no newline at eof.</p> 43 <p>Tests that a cue with no newline at eof is properly parsed.</p>
55 <video> 44 <video>
56 <track src="captions-webvtt/tc003-newlines.vtt" onload="trackLoaded( )"> 45 <track src="captions-webvtt/tc003-no-newline-at-eof.vtt" onload="tra ckLoaded()" default>
57 <track src="captions-webvtt/tc003-no-newline-at-eof.vtt" onload="tra ckLoaded()">
58 </video> 46 </video>
59 </body> 47 </body>
60 </html> 48 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698