Re: Fix for bug 640 - moving thin frames leaves mess

[Alexandre Julliard]

Pavel Roskin  writes:
> The focus rectangle (e.g. the rectangle around active buttons) should be
> painted using the system text color.  Xoring this color with background is
> wrong.  Even on systems where it works, it changes the color from black to
> dark gray (light gray xor white), although not too much to notice it.
> DrawFocusRect() has been changed to use R2_COPYPEN mode.

I don't think that's correct. MSDN explicitly states that
DrawFocusRect uses XOR, so that you can use it a second time to remove
the rectangle.