Bug 24639

Summary: [GTK] GStreamer MediaPlayer is unable to correctly querry duration
Product: WebKit Reporter: Gustavo Noronha (kov) <gustavo>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: cascardo, commit-queue, pnormand
Priority: P2 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
Proposed gst version check
none
Proposed gst version check (with fixed ChangeLog) none

Description Gustavo Noronha (kov) 2009-03-16 23:56:42 PDT
When querying time duration of ogg files (which are the only we support for now) gst_element_query_duration currently returns true, and sets the length to 0. We are adding a work-around to our MediaPlayerPrivate class, but this should be fixed in GStreamer.
Comment 1 Thadeu Lima de Souza Cascardo 2009-03-17 00:10:33 PDT
http://bugzilla.gnome.org/show_bug.cgi?id=575649
Comment 2 Philippe Normand 2009-09-23 09:24:28 PDT
Fix in GStreamer has been released. The workaround we have can now be removed maybe?
Comment 3 Gustavo Noronha (kov) 2009-09-23 15:50:41 PDT
(In reply to comment #2)
> Fix in GStreamer has been released. The workaround we have can now be removed
> maybe?

With a version check, sure =). Want to write a patch for me to review?
Comment 4 Philippe Normand 2009-09-24 00:55:05 PDT
Created attachment 40049 [details]
Proposed gst version check
Comment 5 Philippe Normand 2009-09-24 01:16:48 PDT
Created attachment 40051 [details]
Proposed gst version check (with fixed ChangeLog)
Comment 6 Xan Lopez 2009-09-24 01:21:12 PDT
Comment on attachment 40051 [details]
Proposed gst version check (with fixed ChangeLog)

r=me
Comment 7 WebKit Commit Bot 2009-09-24 03:50:06 PDT
Comment on attachment 40051 [details]
Proposed gst version check (with fixed ChangeLog)

Clearing flags on attachment: 40051

Committed r48713: <http://trac.webkit.org/changeset/48713>
Comment 8 WebKit Commit Bot 2009-09-24 03:50:10 PDT
All reviewed patches have been landed.  Closing bug.