lookupNamespaceURI should probably be aware of xml prefix

RESOLVED FIXED

Status

()

Core
DOM: Core & HTML
--
minor
RESOLVED FIXED
13 years ago
9 years ago

People

(Reporter: WeirdAl, Assigned: WeirdAl)

Tracking

Trunk
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

13 years ago
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... :)
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
namespaces)....
They do have associated namespace URIs.
   xml     http://www.w3.org/XML/1998/namespace
   xmlns   http://www.w3.org/2000/xmlns/
See: http://www.w3.org/TR/xml-names11/#xmlReserved
Note that the lookup algorithm is specified in
http://www.w3.org/TR/2004/REC-DOM-Level-3-Core-20040407/namespaces-algorithms.html#lookupNamespaceURIAlgo
That said, I never understood why that doesn't special-case xml and xmlns. DOM
Level 3 XPath ended up special-casing xml:
http://www.w3.org/TR/2004/NOTE-DOM-Level-3-XPath-20040226/xpath.html#XPathEvaluator-createNSResolver
(Assignee)

Comment 4

12 years ago
Created attachment 199716 [details] [diff] [review]
special-case xml, xmlns prefixes

javascript:alert(document.lookupNamespaceURI("xml") + "\n" +
document.lookupNamespaceURI("xmlns"))
Assignee: general → ajvincent
Status: NEW → ASSIGNED
Attachment #199716 - Flags: superreview?(peterv)
Attachment #199716 - Flags: review?(peterv)
Attachment #199716 - Flags: superreview?(peterv)
Attachment #199716 - Flags: superreview+
Attachment #199716 - Flags: review?(peterv)
Attachment #199716 - Flags: review+
(Assignee)

Comment 5

12 years ago
fixed with checkin by timeless
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Updated

9 years ago
Component: DOM: Core → DOM: Core & HTML
QA Contact: ian → general
You need to log in before you can comment on or make changes to this bug.