[Intel-gfx] [PATCH 2/3] Xv overlay: set default color-key to black
barry.scott at onelan.co.uk
Mon Nov 9 10:04:58 PST 2009
Daniel Vetter wrote:
> On Thu, Nov 05, 2009 at 03:01:20PM -0800, Eric Anholt wrote:
>> On Mon, 2009-10-26 at 14:15 +0100, Daniel Vetter wrote:
>>> This way we avoid the dreaded blue. Furthermore most video players
>>> use black as background, so no one can notice the color-key painting.
>>> Leaking of the overlay outside the window is only possible when moving,
>>> because the kernel should place it pixel-accurate.
>> So, if I drag a window containing black over my XV overlay window, the
>> movie will show through, right?
>> I think some video players actually painted over the color key for
>> interface, as well. This would likely break that, given that black's a
>> common color.
And if an application places a shaped window over the movie window you
get the problem with bleed though if the key color choice is a common
> Yes, you're right. Of course, we can slightly change it to 010101 like we
> do with the blue, but that's gonna look ugly with anti-aliased black
> fonts, sooner or later. Let's just drop this one and remind people (like
> me) with that ugly blue that they should somewhen upgrade their hw to
> something more decent.
On our VIA hardware its 0x000821, which is weird enough to not match in
Using 010101 which is a gray may match a fade to black.
More information about the Intel-gfx