[Webkit-unassigned] [Bug 44375] Copying Paths is expensive. toPathData() should avoid causing a copy.

[bugzilla-daemon at webkit.org]

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





--- Comment #7 from Kenneth Russell <kbr at google.com>  2010-08-23 12:19:06 PST ---
(From update of attachment 65026)
I think that the general direction of making Path RefCounted (https://bugs.webkit.org/show_bug.cgi?id=44402) and reusing the unit circle's path (https://bugs.webkit.org/show_bug.cgi?id=44374) are better than making these changes simply to avoid calling Path's copy constructor and/or assignment operator. Note that with these changes there is still no caching of Paths going on inside of routines like RenderMathMLRoot::paint, GraphicsContext::addRoundedRectClip, etc., and no clear opportunity to do so.

If it seems that making Path RefCounted would be a loss because of additional mallocs/frees during temporary Path construction and destruction, perhaps we could consider making a cross-platform internal data structure in Path which holds the PlatformPathPtr and itself is reference counted.

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