[webkit-reviews] review denied: [Bug 6648] Safari strips-off namespace when using getAttribute() : [Attachment 6231] Patch

[bugzilla-request-daemon at opendarwin.org]

Darin Adler <darin at apple.com> has denied Boris Daljevi&#263;
<boris at backbase.com>'s request for review:
Bug 6648: Safari strips-off namespace when using getAttribute()
http://bugzilla.opendarwin.org/show_bug.cgi?id=6648

Attachment 6231: Patch
http://bugzilla.opendarwin.org/attachment.cgi?id=6231&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
I don't understand this patch.

The getAttribute, setAttribute, and removeAttribute implementations all do the
same thing, calling through to get/set/removeAttributeNS with a namespace URI
of null.

How can it possibly be correct to change only setAttribute and not the other
two?

If you can answer that question, it seems OK to set this to review? again. By
the way, it's best to include the change log entry and the layout test case in
the patch.