[webkit-reviews] review denied: [Bug 10740] Need to remove KCanvas enums from SVG code : [Attachment 10581] First attempt

[bugzilla-request-daemon at opendarwin.org]

Eric Seidel <macdome at opendarwin.org> has denied Eric Seidel
<macdome at opendarwin.org>'s request for review:
Bug 10740: Need to remove KCanvas enums from SVG code
http://bugzilla.opendarwin.org/show_bug.cgi?id=10740

Attachment 10581: First attempt
http://bugzilla.opendarwin.org/attachment.cgi?id=10581&action=edit

------- Additional Comments from Eric Seidel <macdome at opendarwin.org>
This wasn't really what I meant with this bug.	More I meant that the KCanvas
classes should all use the DOM enums instead of having their own duplicate
copies with the KC prefix.  Especially since KCanvas is slated for the chopping
block.

SVGSVGElement *svg = static_cast<SVGSVGElement*>(node());

is not at all safe.  RenderSVGContainers can also represent SVGGElements as
well as SVGSVGElements.