[Webkit-unassigned] [Bug 36564] Performance regression for setting content of <text> in SVG

[bugzilla-daemon at webkit.org]

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





--- Comment #11 from Beth Dakin <bdakin at apple.com>  2010-03-26 15:06:04 PST ---
Here's a summary of what I learned about this performance regression. This
could be helpful in attempting to fix the rest of the regression.

1. clipperBoundingBoxForRenderer() was the major source of the regression. This
patch optimizes that function a lot, but it could probably be optimized more.
There is a lot of overhead associated with getRenderSVGResourceById(). It would
be better if SVGRenderStyle stored a pointer to the resource instead of just
the ID. There should be a way to make sure these are reset whenever an id on
the page changes.

2. It might help to inline clipperBoundingBoxForRenderer(). And
filterBoundingBoxForRenderer() and maskerBoundingBoxForRenderer() while we're
at it. 

3. Being able to cache repaint rects in RenderSVGContainer and RenderSVGRoot
actually would NOT speed up this test case, but it would speed up other tests.
The cached repaint rect would need to be invalidated in layout, and by changing
the text content, this test case lays out so frequently that there would be
nothing to gain from caching repaint rects in these classes.

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