[webkit-reviews] review denied: [Bug 20364] [XBL] Add shadow tree rendering : [Attachment 22759] First try: add rendering code following SVGUseElement example
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Aug 12 16:56:56 PDT 2008
Eric Seidel <eric at webkit.org> has denied Julien Chaffraix
<jchaffraix at webkit.org>'s request for review:
Bug 20364: [XBL] Add shadow tree rendering
https://bugs.webkit.org/show_bug.cgi?id=20364
Attachment 22759: First try: add rendering code following SVGUseElement example
https://bugs.webkit.org/attachment.cgi?id=22759&action=edit
------- Additional Comments from Eric Seidel <eric at webkit.org>
This check looks wrong:
f (templateElement->attached())
tempalteElement is in the xbl doc, no?
this one doens't need braces:
if (node->hasTagName(XBLNames::templateTag)) {
Otherwise looks fine.
r- for the worng check. I don't need to see this again, if you can convince me
over IRC that you've put the right check in instead of the templateElement
thing... well, and I guess you should add a test case too! :)
More information about the webkit-reviews
mailing list