Closed Bug 198433 Opened 22 years ago Closed 22 years ago

"setting a property that has only a getter" error setting text property of OPTION object

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.4alpha

People

(Reporter: shakahshakah, Unassigned)

References

Details

(Keywords: regression)

Attachments

(2 files)

HTML page that shows the problem
660 bytes, text/html
Details
Make nsIDOMNSHTMLOptionElement shadow properties in nsIDOMHTMLOptionElement instead of vise versa.
759 bytes, patch
fabian
: review+
bzbarsky
: superreview+
Details | Diff | Splinter Review
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.4a) Gecko/20030319 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.4a) Gecko/20030319 The JavaScript console shows the error mentioned in the summary when setting the text property of an OPTION object (inside a SELECT object) in the following HTML (error is reported on line 22): <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd"> <html> <head> <title>OPTION Javascript bug</title> </head> <body> <h3>OPTION Javascript bug</h3> <p> <select ID="the_listbox" SIZE=10> <option VALUE="0" SELECTED>default option</option> </select> <script TYPE="text/javascript" LANGUAGE="JavaScript1.2"> <!-- var theListbox = document.getElementById("the_listbox") ; theListbox.length = 10 ; var n ; for(n=0; n<10; ++n) { theListbox.options[n].text = "Item Text " + n ; theListbox.options[n].value = "iv_" + n ; } theListbox.selectedIndex = 5 ; //--> </script> </body> </html> Reproducible: Always Steps to Reproduce: load the HTML mentioned in Details. Actual Results: "setting a property that has only a getter" error in JavaScript console, attributed to line 22. Expected Results: Filled the SELECT listbox with 10 items. Seems to have worked fine until this week's nightlies, though I might be off by a week or so.
sample HTML page
Not JS engine.
Assignee: rogerl → dom_bugs
Component: JavaScript Engine → DOM Core
QA Contact: pschwartau → desale
The fact that it ever worked was a bug. The property is readonly in both DOM1 (http://www.w3.org/TR/1998/REC-DOM-Level-1-19981001/level-one-html.html#ID-70901257) and DOM2 (http://www.w3.org/TR/2003/REC-DOM-Level-2-HTML-20030109/html.html#ID-70901257).
Status: UNCONFIRMED → RESOLVED
Closed: 22 years ago
Resolution: --- → INVALID
Reopening, this is actually a regression. We used to support this, and we intended to still do, that's why nsIDOMNSHTMLOptionElement was created, but that was added to the wrong place in nsDOMClassInfo. Patch coming up.
Status: RESOLVED → UNCONFIRMED
Resolution: INVALID → ---
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Keywords: regression
OS: Windows 2000 → All
Priority: -- → P2
Hardware: PC → All
Target Milestone: --- → mozilla1.4alpha
Comment on attachment 117884 [details] [diff] [review] Make nsIDOMNSHTMLOptionElement shadow properties in nsIDOMHTMLOptionElement instead of vise versa. Add a nice clear comment that the order matters, please?
Attachment #117884 - Flags: superreview+
*** Bug 198424 has been marked as a duplicate of this bug. ***
Attachment #117884 - Flags: superreview?(bzbarsky)
Attachment #117884 - Flags: review?(fabian)
Attachment #117884 - Flags: superreview?(bzbarsky)
Comment on attachment 117884 [details] [diff] [review] Make nsIDOMNSHTMLOptionElement shadow properties in nsIDOMHTMLOptionElement instead of vise versa. woops sorry for not catching that earlier
Attachment #117884 - Flags: review?(fabian) → review+
FIXED.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago → 22 years ago
Resolution: --- → FIXED
Component: DOM: Core → DOM: Core & HTML
QA Contact: desale → general
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: