[webkit-reviews] review granted: [Bug 5737] Implement DOM3 namespace functions : [Attachment 4690] Implement functions

[bugzilla-request-daemon at opendarwin.org]

Eric Seidel <macdome at opendarwin.org> has granted Anders Carlsson
<andersca at mac.com>'s request for review:
Bug 5737: Implement DOM3 namespace functions
http://bugzilla.opendarwin.org/show_bug.cgi?id=5737

Attachment 4690: Implement functions
http://bugzilla.opendarwin.org/attachment.cgi?id=4690&action=edit

------- Additional Comments from Eric Seidel <macdome at opendarwin.org>
We talked about this at length over IRC.

Looks good.  It's nice that we can just copy this directly from the spec...

r=me.  A couple things you could fix before landing:

A couple of the lines could be condensed using:

if (Foo *foo = bar())
    foo->doFoo();

isDefaultNamespace's element functionality should be implemented in terms of
lookupNamespaceURI to prevent repeating ourselves.  (if it has no prefix, check
the namespace, if it does, look it up and check it).

+    if (!namespaceURI().isNull() && namespaceURI() == _namespaceURI &&
+	 !prefix().isNull() && originalElement->lookupNamespaceURI(prefix()) ==
_namespaceURI)
+	 return prefix();

should just be:

if (_namespaceURI.isNull())
    return DOMString();
 if (namespaceURI() == _namespaceURI &&
originalElement->lookupNamespaceURI(prefix()) == _namespaceURI)
    return prefix();