Bug 44619 – drawGDIGlyphs checks if the offset is empty incorrectly causing problems with the shadow

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Alejandro G. Castro]

From the comment:

https://bugs.webkit.org/show_bug.cgi?id=44603#c5

(In reply to comment #2)
> After changing the names I've realized that this condition could be wrong but the old name could make it seem correct:
> 
> +            drawIntoBitmap = !offset.isEmpty() || blur;
> 
> It is in this file: WebCore/platform/graphics/win/FontCGWin.cpp static void drawGDIGlyphs
> 
> Could someone check if in Windows negative offsets of the text actually work, or no offsets but blurring. We could open a new bug in this case.

I’ve asked someone to check, and indeed in <data:text/html,%3Cdiv%20style=%22-webkit-font-smoothing:%20none%22%3E%3Cdiv%20style=%22text-shadow:%202px%202px%20red%22%3Etest%3C/div%3E%3Cdiv%20style=%22text-shadow:%20-2px%20-2px%20red%22%3Etest%3C/div%3E%3C/div%3E> the bottom line is missing the shadow. Good catch!

[Alejandro G. Castro]

Created attachment 65534 [details]
Proposed patch

I can not test the patch but I guess at least it can be used as a initial point.

[Alejandro G. Castro]

landed r66192, closing bug