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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Apr 29 14:09:50 PDT 2009


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.

More information about the webkit-unassigned mailing list