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
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
Depends on:
Blocks: 50918
  Show dependency treegraph
Reported: 2010-12-24 14:23 PST by Philippe Normand
Modified: 2011-02-16 04:09 PST (History)
4 users (show)

See Also:

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

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