[Webkit-unassigned] [Bug 39965] SVG repaintRect should be empty if content got clipped away

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jun 1 03:25:05 PDT 2010


Nikolas Zimmermann <zimmermann at kde.org> changed:

           What    |Removed                     |Added
  Attachment #57487|review?                     |review-
               Flag|                            |

--- Comment #2 from Nikolas Zimmermann <zimmermann at kde.org>  2010-06-01 03:25:05 PST ---
(From update of attachment 57487)
Hi Dirk, some comments:

 +          The reference of the RenderObjects get already applied to the resource
I don't understand this comment.
I guess it's related to the fact that you're creating ClipperData/MaskerData earlier now, but I don't understand why this is needed.

Please remove the extra spaces.

 +      SVGClipPathElement* clipElement = static_cast<SVGClipPathElement*>(node());
That's not needed, you can leave it as it was before. You're not using 'clipElement' anywhere but in the if some lines below.

 +              repaintRect.intersect(masker->resourceBoundingBox(const_cast<RenderObject*>(object)));
No need for the const_cast again, you already have a local variable 'renderer' above.

Okay, not much to fixup, but r-, as long as I don't understand why you have to early-create ClipperData/MaskerData etc I don't see the immediate need to do so, especially not if (selfNeedsLayout()) is true :-)
Please enlighten me.


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

More information about the webkit-unassigned mailing list