[webkit-reviews] review granted: [Bug 129246] -webkit-clip-path wrong offset for clipPath references : [Attachment 340807] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jun 26 14:08:28 PDT 2018

Simon Fraser (smfr) <simon.fraser at apple.com> has granted Dirk Schulze
<krit at webkit.org>'s request for review:
Bug 129246: -webkit-clip-path wrong offset for clipPath references

Attachment 340807: Patch


--- Comment #4 from Simon Fraser (smfr) <simon.fraser at apple.com> ---
Comment on attachment 340807
  --> https://bugs.webkit.org/attachment.cgi?id=340807

View in context: https://bugs.webkit.org/attachment.cgi?id=340807&action=review

> Source/WebCore/rendering/RenderLayer.cpp:4179
> +	      
(renderer(), svgReferenceBox, paintingInfo.paintDirtyRect, context);

Does hasTagName(SVGNames::clipPathTag) mean that the renderer is always a
RenderSVGResourceClipper? (I know the existing code did this.). I would feel
happier with an is<> check, otherwise this code will release-assert.

More information about the webkit-reviews mailing list