Changeset 158979 in webkit
- Timestamp:
- Nov 8, 2013 5:13:29 PM (10 years ago)
- Location:
- trunk
- Files:
-
- 2 added
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r158976 r158979 1 2013-11-08 Eric Carlson <eric.carlson@apple.com> 2 3 getCueAsHTML() on an empty cue should return a document fragment 4 https://bugs.webkit.org/show_bug.cgi?id=124084 5 6 Reviewed by Darin Adler. 7 8 * media/track/track-cue-empty-cue-text-expected.txt: Added. 9 * media/track/track-cue-empty-cue-text.html: Added. 10 1 11 2013-11-08 Alexandru Chiculita <achicu@adobe.com> 2 12 -
trunk/Source/WebCore/ChangeLog
r158978 r158979 1 2013-11-08 Eric Carlson <eric.carlson@apple.com> 2 3 getCueAsHTML() on an empty cue should return a document fragment 4 https://bugs.webkit.org/show_bug.cgi?id=124084 5 6 Reviewed by Darin Adler. 7 8 Test: media/track/track-cue-empty-cue-text.html 9 10 * html/track/WebVTTParser.cpp: 11 (WebCore::WebVTTParser::createDocumentFragmentFromCueText): Don't return early when the 12 passed an empty string. 13 1 14 2013-11-08 Anders Carlsson <andersca@apple.com> 2 15 -
trunk/Source/WebCore/html/track/WebVTTParser.cpp
r158650 r158979 349 349 // 4.8.10.13.5 WebVTT cue text DOM construction rules. 350 350 351 if (!text.length())352 return 0;353 354 351 ASSERT(m_scriptExecutionContext->isDocument()); 355 352 Document* document = toDocument(m_scriptExecutionContext); 356 353 357 354 RefPtr<DocumentFragment> fragment = DocumentFragment::create(*document); 355 356 if (text.isEmpty()) { 357 fragment->parserAppendChild(Text::create(*document, emptyString())); 358 return fragment.release(); 359 } 360 358 361 m_currentNode = fragment; 359 362 m_tokenizer->reset();
Note: See TracChangeset
for help on using the changeset viewer.