[webkit-reviews] review requested: [Bug 15388] RenderSVGContainer (<g>) should not repaint when its bounds change (unless it has a filter) : [Attachment 16563] potential fix

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has asked  for review:
Bug 15388: RenderSVGContainer (<g>) should not repaint when its bounds change
(unless it has a filter)
http://bugs.webkit.org/show_bug.cgi?id=15388

Attachment 16563: potential fix
http://bugs.webkit.org/attachment.cgi?id=16563&action=edit

------- Additional Comments from Eric Seidel <eric at webkit.org>
This seems to dramatically improve lively kernel.  It does not however fix the
attached test case.  The attached test case's behavior is changed, but the new
behavior might actually be an intentional side effect of
absoluteClippedOverflowRect() describing the size of the content of the <svg>
instead of the actual svg's bounds.  I don't know enough about
absoluteClippedOverflowRect's expected behavior to know for certain.

I'm also not sure this fix is 100% ready as I think it may regress Space
Invaders ever so slightly:
http://www.croczilla.com/svg/samples/invaders/invaders.svg

I have not seen any other regressions in my surfing of the SVG web.