[webkit-reviews] review cancelled: [Bug 11625] Investigate possibility to share code between HTMLStyleElement and SVGStyleElement : [Attachment 11547] First attempt

bugzilla-request-daemon at macosforge.org bugzilla-request-daemon at macosforge.org
Sun Nov 19 05:48:26 PST 2006


Rob Buis <rwlbuis at gmail.com> has cancelled Rob Buis <rwlbuis at gmail.com>'s
request for review:
Bug 11625: Investigate possibility to share code between HTMLStyleElement and
SVGStyleElement
http://bugs.webkit.org/show_bug.cgi?id=11625

Attachment 11547: First attempt
http://bugs.webkit.org/attachment.cgi?id=11547&action=edit

------- Additional Comments from Rob Buis <rwlbuis at gmail.com>
This patch should address Darin's issues with the previous patch. Note the
AtomicString changes in particular. I think something like tryGetAttribute(NS)
should either be used everywhere or not at all, ie. it makes little sense IMHO
to limit it to svg use only. Seeing that knowing up front the default value is
not so common, I think we do not need tryGetAttribute(NS).
Cheers,

Rob.



More information about the webkit-reviews mailing list