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

Unified Diff: LayoutTests/media/track/track-cue-empty-crash.html

Issue 22304002: Crash when calling getCueAsHTML() on a TextTrackCue with empty text (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: LayoutTests/media/track/track-cue-empty-crash.html
diff --git a/LayoutTests/media/track/track-cue-empty-crash.html b/LayoutTests/media/track/track-cue-empty-crash.html
new file mode 100644
index 0000000000000000000000000000000000000000..85f0bfdfabfde1c795ddfe84dafdbec2d0378be1
--- /dev/null
+++ b/LayoutTests/media/track/track-cue-empty-crash.html
@@ -0,0 +1,32 @@
+<!DOCTYPE html>
+<html>
+ <head>
+ <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
+
+ <script src=../media-file.js></script>
+ <script src=../video-test.js></script>
+ <script>
+ var fragment;
+ function startTest()
+ {
+ var emptyCue = new TextTrackCue(0, 0, "");
acolwell GONE FROM CHROMIUM 2013/08/05 23:40:32 Should passing null also behave the same way? If s
vcarbune.chromium 2013/08/09 18:47:26 null is converted to the string "null" and further
+ fragment = emptyCue.getCueAsHTML();
+
+ consoleWrite("** The getCueAsHTML() method should return a document fragment **");
+ testExpected("fragment", null, "!=");
+ testExpected("fragment.children.length", 0);
+
+ consoleWrite("");
+ consoleWrite("No crash. PASS.");
+ consoleWrite("");
+
+ endTest();
+ }
+ </script>
+ </head>
+
+ <body onload="startTest()">
+ <p>Tests that having an empty cue does not crash when calling getCueAsHTML().</p>
+ <video controls />
+ </body>
+</html>
« no previous file with comments | « no previous file | LayoutTests/media/track/track-cue-empty-crash-expected.txt » ('j') | Source/core/html/track/WebVTTParser.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698