[Webkit-unassigned] [Bug 23526] clipToImageBuffer() for all platforms

[bugzilla-daemon at webkit.org]

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


eric at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #27361|review+                     |review-
               Flag|                            |




------- Comment #7 from eric at webkit.org  2009-04-29 14:09 PDT -------
(From update of attachment 27361)
I think we can make this more clear.

The problem here is that in CG you can set a mask image on the context, and
then CG will automatically apply it when necessary.

In Cairo, masking is an immediate operation.

GraphicsContext expects masking to function lazily like CG does.

So we have to fix GraphicsContextCairo to save off the mask image, and apply
the mask at the last moment possible (when popping the context).

I think we could do this clearer by saving the mask image on the
GraphicsContextState and applying it during m_data->restore();

You could write a new applyMaskImage() function on m_data which does the actual
apply using the stored image.   the mask image would not inherit between
graphics context states.

Does that seem sane?


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