[Webkit-unassigned] [Bug 58417] SVG object covers CSS background in HTML foreignObject

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Oct 31 09:22:04 PDT 2011


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


Nikolas Zimmermann <zimmermann at kde.org> changed:

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




--- Comment #7 from Nikolas Zimmermann <zimmermann at kde.org>  2011-10-31 09:22:04 PST ---
(From update of attachment 113057)
View in context: https://bugs.webkit.org/attachment.cgi?id=113057&action=review

Looks good to me on first sight, but r- as it duplicates code.
I wonder about the possible implications/side-effects though. I'd prefer if Dave/Simon/Dan could have a look! I'd want to hear their comments too.

> Source/WebCore/rendering/svg/RenderSVGForeignObject.cpp:66
> +    // Paint all phases of FO elements atomically, as though the FO element established its
> +    // own stacking context.

Can't you share this code between InlineBox and RenderSVGFO in a common base?
Replicating this is not a good idea.

-- 
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