Closed Bug 32376 Opened 25 years ago Closed 25 years ago

Populting a SELECT list by setting its length

Categories

(Core :: DOM: Core & HTML, defect, P3)

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: omerk, Assigned: jst)

References

()

Details

Attachments

(1 file)

Cannot populate a SELECT list from another frame. Please see demo.
Confirmed that this works in NS 4.7. -> DOM0
Assignee: rogerl → jst
Component: Javascript Engine → DOM Level 0
QA Contact: rginda → desale
Status: UNCONFIRMED → NEW
Ever confirmed: true
Some clarification: This bug occurs regardless of whether the SELECT list is in a different frame.
Fixing this would be against the DOM Level 2 spec, the "lenght" property on a select element (HTMLSelectElement) is readonly so you should not try to set that property. I think, since "length" is currently not readonly in the HTMLSelectElement definition in mozilla, that the decition to be backwards compatible was made some time ago. Thus this is a valid bug that needs fixing. rods, any comments about this or should I just go ahead and fix this in nsHTMLSelectElement::SetLength()?
Status: NEW → ASSIGNED
OS: Windows NT → All
Hardware: PC → All
*** Bug 32844 has been marked as a duplicate of this bug. ***
Any idea when this is going to be addressed?
I'll try to get this done for M16.
Target Milestone: --- → M16
Updating summbary to reflect the real problem.
Summary: Cross-frame SELECT list population → Populting a SELECT list by setting its length
*** Bug 35593 has been marked as a duplicate of this bug. ***
Attached patch Proposed fix. (deleted) — Splinter Review
rods, could you have a look at the patch I just attached?
This should be fixed now, the testcase works as expected now. Please note that setting the length of a select is against the DOM tho...
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Tested it with 2000-060809. Marking VERIFIED.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: