HTMLCollection with expandos gets confused about element ids

RESOLVED FIXED

Status

()

Core
DOM: Core & HTML
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: djf, Unassigned)

Tracking

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
Created attachment 540751 [details]
An HTML file that demonstrates the bug

If I obtain an HTMLCollection, and then set an expando on it, then set the id of one of the contained elements to match the expando, the HTMLCollection then returns the element rather than the value of the expando when I index it with that name.

But things go wrong when I change the id of that element to something else. Indexing the collection with the expando name now ought to return the expando value, but instead it still returns the element, even though the element no longer has that id.

See the attachment for code that demonstrates the bug.  I've tested it in FF 5 and Aurora 6.0a2 (2011-06-20)

Comment 1

6 years ago
This is a duplicate.
Whiteboard: DUPEME
(Reporter)

Comment 2

6 years ago
Sorry for the dupe.  I didn't see anything like it in Bugzilla, but I haven't figured out how to search Bugzilla well yet.

Would you link to the existing bug?

Comment 3

6 years ago
If I knew the bug# offhand I would have marked this a duplicate...  I'd have to search for it.  In any case, this will go away with the new DOM bindings.

Updated

6 years ago
Depends on: 648801
Bug 648801 fixes this somewhat, except that it favors expandos over contained elements to match the WebIDL spec (see http://www.w3.org/TR/WebIDL/#dfn-named-property-visibility in particular step 4).
Although I guess technically objects that don't have a name creator/setter operation shouldn't support setting of expandos.

Comment 6

6 years ago
No, I think expandos should work....  In particular, setting anything outside the set of supported names should just set an expando, no?

Comment 7

6 years ago
So is this fixed now that bug 648801 is fixed?
(Reporter)

Comment 8

6 years ago
Works for me in Nightly. (But as peterv points out, the behavior has changed to track WebIDL, and expandos always always shadow named elements.)  

Setting status to  RESOLVED FIXED
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED

Updated

4 years ago
Whiteboard: DUPEME
You need to log in before you can comment on or make changes to this bug.