Bug 51602 - [GStreamer] wrong media duration report in media/controls-after-reload.html
Summary: [GStreamer] wrong media duration report in media/controls-after-reload.html
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 50918
  Show dependency tree / graph
 
Reported: 2010-12-24 14:23 PST by Philippe Normand
Modified: 2011-02-16 04:09 PST (History)
4 users (show)

See Also:

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+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Philippe Normand 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.
Comment 1 Philippe Normand 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
Comment 2 Philippe Normand 2010-12-28 06:49:41 PST 
Comment on attachment 77552 [details]
subparse: WebVTT parsing support

Heh wrong bug tracker :)
Comment 3 Philippe Normand 2011-01-27 03:51:28 PST 
media/controls-styling.html and media/video-volume-slider.html are affected by this bug as well.
Comment 4 Philippe Normand 2011-01-27 03:55:01 PST 
^^ skipped in r76783
Comment 5 Philippe Normand 2011-02-15 07:52:55 PST 
Created attachment 82454 [details]
proposed patch
Comment 6 Martin Robinson 2011-02-15 09:09:28 PST 
Comment on attachment 82454 [details]
proposed patch

Great stuff.
Comment 7 Philippe Normand 2011-02-16 03:10:53 PST 
Committed r78689: <http://trac.webkit.org/changeset/78689>
Comment 8 WebKit Review Bot 2011-02-16 04:09:46 PST 
http://trac.webkit.org/changeset/78689 might have broken GTK Linux 64-bit Debug