[Webkit-unassigned] [Bug 20699] SVGPaintServer CleanUp

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Sep 23 12:53:47 PDT 2008


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


vbs85 at gmx.de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #23712|                            |review?(eric at webkit.org)
               Flag|                            |




------- Comment #33 from vbs85 at gmx.de  2008-09-23 12:53 PDT -------
Created an attachment (id=23712)
 --> (https://bugs.webkit.org/attachment.cgi?id=23712&action=view)
SVGPaintServer cleanUp - pre fifth part

This only a patch to discuss the next steps. I want to show what I think we
could do next.

I creat a new ImageBuffer to have a new GraphicsContext (save and restore could
be necessary for qt) to draw on.
I add the given path to the context, call applyStrokeStyleToContext() and call
strokeBoundingBox() on the GraphicsContext. Every platform has its own
strokeBoundingBox() implementation to create the boundingbox of a stroked path.
This should work for all platforms (don't know how to do it on Qt at the
moment).
Skia in Chromium seems to do it as well but duplicate some functions like
boundingboxes for normal paths.
After this changes, we can move strokeBBox() to RenderPath.cpp.

The old implementation seems not to create a new context on every call and try
to use an existing one, right? I tried something similar, it worked for Cg and
Cairo but qt seems to delete the QOainter to early.


-- 
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