Last Comment Bug 246604 - lookupNamespaceURI should probably be aware of xml prefix
: lookupNamespaceURI should probably be aware of xml prefix
Product: Core
Classification: Components
Component: DOM: Core & HTML (show other bugs)
: Trunk
: x86 All
-- minor (vote)
: ---
Assigned To: Alex Vincent [:WeirdAl]
: Andrew Overholt [:overholt]
Depends on:
  Show dependency treegraph
Reported: 2004-06-13 15:58 PDT by Alex Vincent [:WeirdAl]
Modified: 2008-07-31 02:39 PDT (History)
5 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

special-case xml, xmlns prefixes (1.29 KB, patch)
2005-10-15 23:26 PDT, Alex Vincent [:WeirdAl]
peterv: review+
peterv: superreview+
Details | Diff | Splinter Review

Description User image Alex Vincent [:WeirdAl] 2004-06-13 15:58:05 PDT
The xml attribute prefix (for example, xml:lang) is fairly well-known, but 
Mozilla's lookupNamespaceURI method doesn't know about it.  Thus, to call 
aNode.lookupNamespaceURI("xml") would require that we explicitly declare the 
XML namespace.

We have a similar problem with aNode.lookupNamespaceURI("xmlns").  These are 
namespaces that are not going to change, and our DOM code shouldn't have to 
walk up the tree of nodes to find the right values... :)
Comment 1 User image Boris Zbarsky [:bz] (still a bit busy) 2004-07-13 02:00:58 PDT
This is a performance enhancement, right?  Since "xmlns" and "xml" don't have
associated namespace URIs (due to the fact that they are not actually
Comment 2 User image Hixie (not reading bugmail) 2004-07-13 02:18:00 PDT
They do have associated namespace URIs.
Comment 3 User image Peter Van der Beken [:peterv] 2004-07-13 03:25:11 PDT
Note that the lookup algorithm is specified in
That said, I never understood why that doesn't special-case xml and xmlns. DOM
Level 3 XPath ended up special-casing xml:
Comment 4 User image Alex Vincent [:WeirdAl] 2005-10-15 23:26:05 PDT
Created attachment 199716 [details] [diff] [review]
special-case xml, xmlns prefixes

javascript:alert(document.lookupNamespaceURI("xml") + "\n" +
Comment 5 User image Alex Vincent [:WeirdAl] 2005-11-30 19:31:03 PST
fixed with checkin by timeless

Note You need to log in before you can comment on or make changes to this bug.