[Webkit-unassigned] [Bug 48516] GraphicsContext: Remove "current path" and have strokePath, fillPath and clipPath take a Path argument

[bugzilla-daemon at webkit.org]

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





--- Comment #4 from Andreas Kling <kling at webkit.org>  2010-10-28 23:04:07 PST ---
(In reply to comment #2)
> In my opinion, a bug in the implementation is not good justification for changing the API.

You're right, the primary reason I wanted to do this was to get a cleaner, more concise API.

(In reply to comment #3)
> This way we would save the operation of re-adding the same path onto the context for a fill+stroke operation of shapes in SVG.

You're right, I didn't properly consider the fill+stroke redundancy since that doesn't happen on Qt.

New suggestion: how about replacing beginPath()+addPath() with setPath(const Path&) (since that's how it's used anyway) and not expecting strokePath() or fillPath() to touch the current path?

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