[Webkit-unassigned] [Bug 68251] clipPath does not work on foreignObject
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Nov 9 02:47:00 PST 2011
https://bugs.webkit.org/show_bug.cgi?id=68251
Dirk Schulze <krit at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #114155|review? |review-
Flag| |
--- Comment #5 from Dirk Schulze <krit at webkit.org> 2011-11-09 02:47:00 PST ---
(From update of attachment 114155)
View in context: https://bugs.webkit.org/attachment.cgi?id=114155&action=review
It looks good for me in general, I r- it because not only clip-path is effected. I'd like to see masked and filtered FO examples as well. And one example for opacity.
> Source/WebCore/ChangeLog:6
> + clipPath does not work on foreignObject
> + https://bugs.webkit.org/show_bug.cgi?id=68251
> +
> + Reviewed by NOBODY (OOPS!).
The change log needs to be more details. What have you done and why? This should be included in the change log.
> Source/WebCore/rendering/svg/RenderSVGForeignObject.cpp:62
> if (opacity < 1.0f)
> childPaintInfo.context->beginTransparencyLayer(opacity);
I think we do the opacity in SVGRenderSupport as well. Can you remove the calls here and after your code please? Also, can you add a test to verify the behavior please?
> Source/WebCore/rendering/svg/RenderSVGForeignObject.cpp:75
> if (opacity < 1.0f)
> childPaintInfo.context->endTransparencyLayer();
(needs to be removed as well)
--
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