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

Unified Diff: LayoutTests/media/track/track-webvtt-tc032-degenerate-cues.html

Issue 75273002: Handle recovery on "timestamp-looking" lines in the WebVTT parser (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: LayoutTests/media/track/track-webvtt-tc032-degenerate-cues.html
diff --git a/LayoutTests/media/track/track-webvtt-tc030-interspersed-non-cue.html b/LayoutTests/media/track/track-webvtt-tc032-degenerate-cues.html
similarity index 63%
copy from LayoutTests/media/track/track-webvtt-tc030-interspersed-non-cue.html
copy to LayoutTests/media/track/track-webvtt-tc032-degenerate-cues.html
index ab8d206623917a4344ff6ff005583c7ec2889840..9fc7f742e7112d5b41e613578e84e3c6abc7668c 100644
--- a/LayoutTests/media/track/track-webvtt-tc030-interspersed-non-cue.html
+++ b/LayoutTests/media/track/track-webvtt-tc032-degenerate-cues.html
@@ -22,19 +22,24 @@
findMediaElement();
var expected =
{
- length: 2,
+ length: 3,
tests:
[
{
+ property: "startTime",
+ values: [ "0", "2", "4" ],
+ },
+ {
+ property: "endTime",
+ values: [ "1", "3", "5" ],
+ },
+ {
property: "text",
- values:
- [
- 'First',
- 'Second',
- ],
+ values: [ '', '', '' ],
},
],
};
+
testCues(0, expected);
allTestsEnded();
@@ -42,9 +47,9 @@
</script>
</head>
<body onload="enableAllTextTracks()">
- <p>Tests that an empty line after an identifier line discards the current cue and restarts the cue loop.</p>
+ <p>Tests that a degenerate cues without separating blank lines are parsed correctly.</p>
<video>
- <track src="captions-webvtt/tc030-interspersed-non-cue.vtt" onload="trackLoaded()">
+ <track src="captions-webvtt/tc032-degenerate-cues.vtt" onload="trackLoaded()">
</video>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698