[Webkit-unassigned] [Bug 51928] [chromium] Enable reuse of RenderSurface for drawing.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=51928





--- Comment #10 from W. James MacLean <wjmaclean at chromium.org>  2011-01-07 06:59:07 PST ---
(In reply to comment #8)
> This change seems to violate the comment above it:
> 
>     // Mask out writes to alpha channel: subpixel antialiasing via Skia results in invalid
>     // zero alpha values on text glyphs. The root layer is always opaque.
>     m_context->colorMask(true, true, true, false);
> 
> Either this change is wrong, or the comment is wrong.  Vangelis, what do you think?

I've moved the comment along with the change to the color mask, but I do not know about the origin of the comment.

I'm happy to prepare a test-patch people could apply to ToT to see for themselves the difference between leaving the alpha mask on and off, if that would help. As mentioned earlier, so far it's just the one test case I know of that triggers it.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.