[webkit-reviews] review denied: [Bug 18473] Animation <use> fixes : [Attachment 20522] <use> fixes

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has denied Antti Koivisto <koivisto at iki.fi>'s
request for review:
Bug 18473: Animation <use> fixes
http://bugs.webkit.org/show_bug.cgi?id=18473

Attachment 20522: <use> fixes
http://bugs.webkit.org/attachment.cgi?id=20522&action=edit

------- Additional Comments from Eric Seidel <eric at webkit.org>
I prefer arguments to have more informative names than "b":
setInstanceUpdatesBlocked
"blockUpdates" perhaps?

Is gElementsWithInstanceUpdatesBlocked the official pattern now? I thought s_
was?

 void SVGUseElement::removeDisallowedElementsFromSubtree(Node* element)

needs a comment as to why RefPtr<Node> is not required to protect against
MutationEvent listeners doing bad things.

+bool SVGSMILElement::isSMILElement(Node* node)
I'm surprised that's even needed.  I guess I would have figured there would
have been a virtual bool isSVGSMILElement() const { return false; } on
SVGElement.

r- due to the possible crasher due to mutation events (or lack of comment)