Bug 51602

Summary: [GStreamer] wrong media duration report in media/controls-after-reload.html
Product: WebKit Reporter: Philippe Normand <pnormand>
Component: MediaAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric.carlson, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 50918    
Attachments:
Description Flags
subparse: WebVTT parsing support
none
proposed patch mrobinson: review+

Philippe Normand
Reported 2010-12-24 14:23:27 PST
On 32-bits Release a null duration is reported instead of 6 seconds. This affects the rendering of the controls since we now display the media duration there. Will skip the test for now.
Attachments
subparse: WebVTT parsing support (27.24 KB, text/plain)
2010-12-28 06:49 PST, Philippe Normand 		no flags
Details
proposed patch (5.49 KB, patch)
2011-02-15 07:52 PST, Philippe Normand 		mrobinson: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Philippe Normand
Comment 1 2010-12-28 06:49:05 PST
Created attachment 77552 [details] subparse: WebVTT parsing support WebVTT is a new subtitle format for HTML5 video. In this firtst version of the parser the cue settings are parsed but only stored in the internal parser state structure. Later on these settings could be part of the GstBuffer metadata. Fixes bug #629764
Philippe Normand
Comment 2 2010-12-28 06:49:41 PST
Comment on attachment 77552 [details] subparse: WebVTT parsing support Heh wrong bug tracker :)
Philippe Normand
Comment 3 2011-01-27 03:51:28 PST
media/controls-styling.html and media/video-volume-slider.html are affected by this bug as well.
Philippe Normand
Comment 4 2011-01-27 03:55:01 PST
^^ skipped in r76783
Philippe Normand
Comment 5 2011-02-15 07:52:55 PST
Created attachment 82454 [details] proposed patch
Martin Robinson
Comment 6 2011-02-15 09:09:28 PST
Comment on attachment 82454 [details] proposed patch Great stuff.
Philippe Normand
Comment 7 2011-02-16 03:10:53 PST
Committed r78689: <http://trac.webkit.org/changeset/78689>
WebKit Review Bot
Comment 8 2011-02-16 04:09:46 PST
http://trac.webkit.org/changeset/78689 might have broken GTK Linux 64-bit Debug
Note You need to log in before you can comment on or make changes to this bug.