Bug 130532 – Add a flushing mechanism for the WebVTTParser

• Status: RESOLVED
• Resolution: FIXED
• Component: Media

[Brent Fulgham]

1. Use TextResourceDecoder for the WebVTTParser
	1ac2b3081492ab8661429230004013a9a3623f0f
	https://codereview.chromium.org/54203003
	
	[e-mail]
	
2. Add a flushing mechanism for the WebVTTParser
	24343d32bc36d20d59ef3e10040faddde65929e6
	https://codereview.chromium.org/59453002
	
	[e-mail]

3. Don't find(':') twice in WebVTTParser::collectMetadataHeader
	ef2793f1d9d207389589ecf9a136ea5c7170b4af
	https://codereview.chromium.org/75233002
	
	[e-mail]

4. Decouple tokenizer/tree builder state from WebVTTParser
	a29f2f8324abcd4826b41e7dc34373dee2ec57a9
	https://codereview.chromium.org/52713005
	
	[e-mail]

5. Handle buffer boundaries in the WebVTT parser
	7ce003c8d47b39f116f1f2a592a5dfb4a9e77228
	https://codereview.chromium.org/64303004
	
	[e-mail]

6. Fix tag closing issues in the WebVTT tree builder
	18f896b3498478311e880f782813d5dfc8c0c7d1
	https://codereview.chromium.org/96933004
	
	[e-mail]

7. WebVTT only allows <rt> if the currently innermost node is <ruby>
	8670e8ecb13254a651f5493ec83f4484d18c154e
	https://codereview.chromium.org/98783005
	
	[e-mail]

8. Use the return value to signal success for VTTParser helpers
	4ac55780a6af3d76e0159c1d145330964ad56647
	https://codereview.chromium.org/104713002
	
	[e-mail]

[Radar WebKit Bug Importer]

<rdar://problem/16382589>

[Brent Fulgham]

Created attachment 227326 [details]
Patch

[Eric Carlson]

Comment on attachment 227326 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=227326&action=review

> Source/WebCore/ChangeLog:64
> [phone]  Brent Fulgham  <[e-mail]>
> +
> +        Need a short description (OOPS!).
> +        Need the bug URL (OOPS!).
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        Tests: http/tests/media/track/track-webvtt-slow-loading-2.html
> +               http/tests/media/track/track-webvtt-slow-loading.html
> +
> +        * CMakeLists.txt:
> +        * Configurations/FeatureDefines.xcconfig:
> +        * GNUmakefile.list.am:
> +        * WebCore.vcxproj/WebCore.vcxproj:
> +        * WebCore.vcxproj/WebCore.vcxproj.filters:
> +        * WebCore.xcodeproj/project.pbxproj:
> +        * html/track/BufferedLineReader.cpp: Added.
> +        * html/track/BufferedLineReader.h: Added.
> +        * html/track/TextTrackRegion.cpp:
> +        * html/track/VTTCue.cpp:
> +        * html/track/WebVTTParser.cpp:
> +        * html/track/WebVTTParser.h:
> +        * loader/TextTrackLoader.cpp:

Oops!

[Brent Fulgham]

Committed r165997: <http://trac.webkit.org/changeset/165997>