Crash when calling getCueAsHTML() on a TextTrackCue with empty text

Source/core/html/track/WebVTTParser.cpp

Index: Source/core/html/track/WebVTTParser.cpp
diff --git a/Source/core/html/track/WebVTTParser.cpp b/Source/core/html/track/WebVTTParser.cpp
index 4a7b524f5922c12f1894f8bc1d2e4719f9ec68e1..e556b799b9a793f6626d7ddb6c2ebb7f166e81ae 100644
--- a/Source/core/html/track/WebVTTParser.cpp
+++ b/Source/core/html/track/WebVTTParser.cpp
@@ -328,9 +328,6 @@ PassRefPtr<DocumentFragment>  WebVTTParser::createDocumentFragmentFromCueText(co
     // 4.8.10.13.4 WebVTT cue text parsing rules and
     // 4.8.10.13.5 WebVTT cue text DOM construction rules.
 
-    if (!text.length())
-        return 0;

[acolwell GONE FROM CHROMIUM, 2013/08/05 23:40:32]

nit: Do you really need to jump though all the hoops below. Can't you just move
this down 2 lines and do 'return DocumentFragment::create(document);'?

[vcarbune.chromium, 2013/08/09 18:47:26]

Yes, you're right, it shouldn't go through everything below. I also gave this a
bit more thought and I think what it should actually be returned is a
DocumentFragment with an empty Text as child node (empty string). This makes it
consistent with the regular behaviour.

-
     ASSERT(m_scriptExecutionContext->isDocument());
     Document* document = toDocument(m_scriptExecutionContext);