After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 721824 - Query duration and position from source element
Query duration and position from source element
Status: RESOLVED FIXED
Product: rygel
Classification: Applications
Component: librygel-renderer
git master
Other Linux
: Normal normal
: ---
Assigned To: rygel-maint
rygel-maint
Depends on:
Blocks:
 
 
Reported: 2014-01-08 19:52 UTC by Parthiban Balasubramanian
Modified: 2014-06-14 12:48 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Query duration and position from source element (1.41 KB, patch)
2014-01-08 19:52 UTC, Parthiban Balasubramanian
committed Details | Review
renderer-gst: Update video duration (1.17 KB, patch)
2014-06-10 15:44 UTC, Jens Georg
committed Details | Review
renderer-gst: Use duration from meta-data (2.79 KB, patch)
2014-06-14 12:47 UTC, Jens Georg
none Details | Review

Description Parthiban Balasubramanian 2014-01-08 19:52:54 UTC
Created attachment 265749 [details] [review]
Query duration and position from source element

This change queries duration and position from the playbin's source element instead of directly from playbin.
This fixes DLNA CTT 7.3.157.8.
Comment 1 Jens Georg 2014-01-09 08:09:39 UTC
Review of attachment 265749 [details] [review]:

+1
Comment 2 Jussi Kukkonen 2014-04-30 08:49:28 UTC
Damn, this seems to break at least with some element/gstreamer version combo. I'm playing a mp3 and the source element exists but its position stays at 0 (while the position in playbin at least changes, not entirely sure if it's a useful value for seeking).

I'll debug a bit further and file a new bug.
Comment 3 Jens Georg 2014-06-10 15:06:40 UTC
I suppose it breaks as the normal soupsrc can't possibly know the duration.

dlnasrc can, but that's not the fix. I'm reverting the commit and re-opening the bug.
Comment 4 Jens Georg 2014-06-10 15:14:47 UTC
This actually seems to be a non-bug with GStreamer >= 1.2.4, iow works fine for me when reverted.
Comment 5 Jens Georg 2014-06-10 15:36:09 UTC
Ok, duration is still broken for videos. Wonder why that is...
Comment 6 Jens Georg 2014-06-10 15:44:22 UTC
Created attachment 278212 [details] [review]
renderer-gst: Update video duration

On READY->PAUSED state change, check for duration availability and signalize
it.

Signed-off-by: Jens Georg <mail@jensge.org>
Comment 7 Jens Georg 2014-06-10 16:04:23 UTC
Comment on attachment 278212 [details] [review]
renderer-gst: Update video duration

Attachment 278212 [details] pushed as 0c32705 - renderer-gst: Update video duration
Comment 8 Jens Georg 2014-06-14 12:47:52 UTC
Created attachment 278441 [details] [review]
renderer-gst: Use duration from meta-data

Until we get the proper duration from the pipeline, use the one supplied in
the meta-data if available.

Signed-off-by: Jens Georg <mail@jensge.org>