Closed Bug 644192 Opened 11 years ago Closed 11 years ago
Firefox 4: Dynamically-created SELECT boxes have no OPTION tags, throw NS
_ERROR _DOM _INDEX _SIZE _ERR error
I am able to get the OPTION tags to render if the SELECT tag is rendered first, then the OPTION tags are added to it.
This broke between Alpha 4 (http://hg.mozilla.org/mozilla-central/rev/231cc7025b6f) <-> 5 (http://hg.mozilla.org/mozilla-central/rev/b2f7bc2aa0bc). Nice if sombody can narrow down further using e.g. http://harthur.github.com/mozregression/
Severity: major → normal
Product: Firefox → Core
QA Contact: general → general
So... The page has been changed so the problem can't be reproduced anymore. Adam, do you have a page that shows the problem?
Nevemind; I got in by spoofing the UA. But I don't see the error from comment 0. Is the site catching it and eating the exception?
XtC, thanks for the prelim range, that sped things up a lot. Last good nightly: 2010-05-23 First bad nightly: 2010-05-24 Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=37b7ca43fafe&tochange=da9e50cb4091 The only <select> related change I could see is: Mounir Lamouri — Bug 551846. <select> size should default to 4 when 'multiple' attribute is present. r=smaug, sr=jst (Note this is just a guess, there are other changesets in the range) Provisionally CCing Mounir, any ideas about the range, could bug 551846 be the cause?
Status: UNCONFIRMED → NEW
Ever confirmed: true
The issue is that mz_() in https://25live.collegenet.com/backbase/3_3_1d/bpc/bpc_02.js tries to do |x.size = 0| where |x| is an HTMLSelectElement. Per the current HTML5 spec text, which Firefox 4 implements, this is supposed to throw. See bug 640800 and bug 639175. This will be changed in an upcoming release, and a request made to change the spec, but in the meantime the workaround is to not set the size of a <select> to 0 (or to catch the exception when doing so)....
Note that I have no idea what mz_() is _actually_ doing since it's all obfuscated. I'm just observing us land in nsHTMLSelectElement::SetSize with aValue == 0 and the caller is that function.
Sorry about the example URL being blocked. I'm in the process of opening up a new URL that will remain unblocked to FF4. And yes, the obfuscated code of our GUI framework makes my life a nightmare. But thanks to all for your help! I will note as soon as the new example URL is available.
Example URL has been edited. There are five hidden SELECT boxes when the page initially loads. Clicking "More Search Options" attempts to replace each SELECT with a new one that has content. All of them trigger an error.
This should be fixed in current trunk.
Status: NEW → RESOLVED
Closed: 11 years ago
OS: Windows 7 → All
Hardware: x86 → All
Resolution: --- → FIXED
It should be fixed with Firefox 4.0.1 (next minor release).
You need to log in before you can comment on or make changes to this bug.