11000 - getElementsByTagName(someHTMLelement) returns nothing in XUL docs

Status: VERIFIED INVALID

(Core :: XUL, defect, P3)

Description

[Ben Goodger (use ben at mozilla dot org for email)]

in a XUL document (file URLs are below, they're a modified editor image
properties dialog) getElementsByTagName() returns a zero length collection for
any HTML tag argument, e.g. document.getElementsByTagName("input"); even if
there are input elements in the doc.

for XUL elements (e.g document.getElementsByTagName("titledbutton")) it returns
a full collection correctly.

Files:
(in function DLGrabInputs())
http://homepages.ihug.co.nz/~rgoodger/lizard/EdImageProps.js
http://homepages.ihug.co.nz/~rgoodger/lizard/EdImageProps.xul
to see the problem in action, put these in chrome://editor/content/default and
load editor, then click the "image" button on the toolbar, and watch the
console.

Comment 1

[Ben Goodger (use ben at mozilla dot org for email)]

I updated the files somewhat. Now, when loaded, the script does not recognise
any HTML elements (giving an error relating to a TR element that's not been
grabbed properly - <TR ID="AdvancedRow">..).. while the XUL titledbutton
called "Advanced" responds to script as expected (goes in and out of pressed
state when clicked).

Comment 2

[Chris Waterson]

HTML elements _always_ return their tag names as UPPERCASE. So, changing your
code to look for upper case HTML tags (e.g., "INPUT", "BUTTON") should allow
you to work around this problem.

I'm loathe to put case-insensitive compares in the XUL document's
implementation of getElementsByTagName(), as it would preclude us from
differentiating between case-sensitive XML tags. DOM Level 1 (sec 1.1.6) seems
to indicate that we are doing the right thing here.

Marking INVALID for now; hyatt, rgoodger, re-open if you find evidence that
indicates that this should be otherwise. (Hyatt, we should add this to the XUL
FAQ, I think...)

Comment 3

[ckritzer (gone)]

Assigning to originator per new organizational structure.

Comment 4

[ckritzer (gone)]

Changing QA Contact back to [e-mail]

Comment 5

[ckritzer (gone)]

Marking VERIFIED INVALID per waterson's comments.

Comment 6

[leger]

BULK MOVE: Changing component from XUL to XP Toolkit/Widgets: XUL.  XUL 
component will be deleted.