Bug 24651

Summary: Skia does not always render text fill/stroke pattern/gradient/color correctly
Product: WebKit Reporter: Scott Violet <sky>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: brettw
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   
Attachments:
Description Flags
Fix for 24651 eric: review+

Description Scott Violet 2009-03-17 13:12:56 PDT
Skia's text rendering gives preference to the gradient, then the pattern. This is problematic though in that once a gradient has been set the gradient getter always returns non-null. Skia needs to only use the gradient/pattern if the fill/strokecolorspace indicates the gradients/pattern should be used.

This can be seen when using the test case LayoutTests/fast/canvas/canvas-text-alignment.html.
Comment 1 Scott Violet 2009-03-17 13:17:23 PDT
Created attachment 28697 [details]
Fix for 24651
Comment 2 Eric Seidel (no email) 2009-03-17 14:06:36 PDT
Comment on attachment 28697 [details]
Fix for 24651

OK.
Comment 3 Eric Seidel (no email) 2009-03-17 15:17:45 PDT
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/platform/graphics/GraphicsContext.cpp
	M	WebCore/platform/graphics/GraphicsContext.h
	M	WebCore/platform/graphics/GraphicsContextPrivate.h
	M	WebCore/platform/graphics/skia/SkiaFontWin.cpp
Committed r41789