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 594623 - Expose playsink as a fully-fledged element
Expose playsink as a fully-fledged element
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal blocker
: 0.10.25
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-09-09 11:28 UTC by Edward Hervey
Modified: 2009-09-14 10:46 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
playback: Register playsink as an element. (9.03 KB, patch)
2009-09-09 11:33 UTC, Edward Hervey
committed Details | Review
GstPlaySink: Expose flags as a gobject property. (3.13 KB, patch)
2009-09-09 11:34 UTC, Edward Hervey
committed Details | Review
GstPlaySink: Expose 'reconfigure' as an action signal. (2.03 KB, patch)
2009-09-09 11:34 UTC, Edward Hervey
committed Details | Review
playsink: Expose mute,volume,vis-plugin and font-desc properties (3.73 KB, patch)
2009-09-14 10:18 UTC, Edward Hervey
committed Details | Review

Description Edward Hervey 2009-09-09 11:28:31 UTC
playsink isn't available from outside the playback plugin.

Having it available as a fully-fledged element would make it easier to create high-level elements like playbin.
Comment 1 Edward Hervey 2009-09-09 11:33:30 UTC
Created attachment 142777 [details] [review]
playback: Register playsink as an element.

This allows using playsink from outside the playback plugin.

Add code to be able to request the sink pads using standard GStreamer API.

TODO : expose GObject properties/signals.
Comment 2 Edward Hervey 2009-09-09 11:34:00 UTC
Created attachment 142778 [details] [review]
GstPlaySink: Expose flags as a gobject property.
Comment 3 Edward Hervey 2009-09-09 11:34:15 UTC
Created attachment 142779 [details] [review]
GstPlaySink: Expose 'reconfigure' as an action signal.
Comment 4 Sebastian Dröge (slomo) 2009-09-09 13:13:27 UTC
Good idea, do it :)

Will you also expose the "audio-sink" etc stuff as GObject properties to set custom sinks?
Comment 5 Edward Hervey 2009-09-09 13:14:34 UTC
yes, that's the last thing I have to expose. Just wanted to make it publicly viewable before I push those commits :)
Comment 6 Edward Hervey 2009-09-12 08:51:47 UTC
I *absolutely* want to get this is for the upcoming -base release. I thought I'd have enough time to fix this before the release... but due to the unexpected shift in releases... it'll have to be done faster :(

Marking this as blocker.
Comment 7 Edward Hervey 2009-09-14 10:18:54 UTC
Created attachment 143141 [details] [review]
playsink: Expose mute,volume,vis-plugin and font-desc properties
Comment 8 Edward Hervey 2009-09-14 10:20:54 UTC
After going over playbin2's usage of playsink... the remaining properties (*-sink, last-frame) use a lot of code which is in playbin2.

My feeling at this point... is that I don't want to do these radical changes for this release but still have playsink exposed. We can still add those remaining properties after the release.

Any objections ?
Comment 9 Edward Hervey 2009-09-14 10:46:51 UTC
commit b565a3088c9e51346a05b2dd69e68483fe2f2bda
Author: Edward Hervey <bilboed@bilboed.com>
Date:   Mon Sep 14 12:18:33 2009 +0200

    playsink: Expose mute,volume,vis-plugin and font-desc properties
    
    https://bugzilla.gnome.org/show_bug.cgi?id=594623

commit 377e781ad57e1edbd575227671dfbb808d4b7114
Author: Edward Hervey <bilboed@bilboed.com>
Date:   Wed Sep 9 12:42:04 2009 +0200

    GstPlaySink: Expose 'reconfigure' as an action signal.

commit bafe9082fdfabe749b837e8d945cba4b555802f6
Author: Edward Hervey <bilboed@bilboed.com>
Date:   Wed Sep 9 11:17:28 2009 +0200

    GstPlaySink: Expose flags as a gobject property.

commit 02c50388c4dd435a601758af4da8bfb70e5836e0
Author: Edward Hervey <bilboed@bilboed.com>
Date:   Tue Sep 8 11:35:20 2009 +0200

    playback: Register playsink as an element.
    
    This allows using playsink from outside the playback plugin.
    
    Add code to be able to request the sink pads using standard GStreamer API.
    
    TODO : expose GObject properties/signals.