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 635800 - xvimagesink flashes black when going from READY_TO_NULL
xvimagesink flashes black when going from READY_TO_NULL
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
unspecified
Other Linux
: Normal normal
: 0.10.33
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-11-25 18:19 UTC by Erik
Modified: 2011-04-05 08:38 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Erik 2010-11-25 18:19:21 UTC
gst_xvimagesink_xwindow_clear() is painting the
window black when going from READY_TO_NULL. It's something we should make
optional I think (and remove at some point). Its also inconsistent as apps would
need to make their video window black initially if they want it to be black.
Comment 1 Sebastian Dröge (slomo) 2010-12-18 13:52:11 UTC
What would you propose as solution? A property to select the color that should be used when going back to NULL *and* in the beginning? Or a property to disable this behaviour completely?
Comment 2 Stefan Sauer (gstreamer, gtkdoc dev) 2010-12-19 18:47:32 UTC
Probably disable drawing via option now and drop it in 0.11. We don't paint the recange black initially, so we should not do it later too.
Comment 3 Jan Schmidt 2010-12-19 20:07:46 UTC
I don't see the point in a property, just remove the behaviour. We want to get back to NULL as quickly as possible, right?
Comment 4 Stefan Sauer (gstreamer, gtkdoc dev) 2010-12-20 08:44:39 UTC
(In reply to comment #3)
> I don't see the point in a property, just remove the behaviour. We want to get
> back to NULL as quickly as possible, right?

That'd be fine for my, but it means that for apps that don't paint one sees the colorkey now, But if noone object, lets just remove it.
Comment 5 Tobias Mueller 2011-04-04 09:38:59 UTC
So apparently nobody objected (at least here in the bug report). Hence I am setting to NEW. Feel free to change if you don't think it's appropriate.
Comment 6 Stefan Sauer (gstreamer, gtkdoc dev) 2011-04-05 08:38:28 UTC
commit 38aba7853eb3027489fc2b67bafc9f4db6b89b39
Author: Stefan Kost <ensonic@users.sf.net>
Date:   Tue Apr 5 11:32:52 2011 +0300

    xvimagesink: don't paint the window black when going to NULL
    
    Leave dealing with the appearance of the window when we are not playing to the
    applications. We anyway want to go to NULL as quickly as possible.
    
    Fixes #635800