Closed Bug 279682 Opened 20 years ago Closed 20 years ago

###!!! ASSERTION: Form control has a frame, but it's not a form frame

Categories

(Core :: Layout: Form Controls, defect)

x86
Linux
defect
Not set
trivial

Tracking

()

RESOLVED FIXED

People

(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)

Details

(Keywords: testcase)

Attachments

(3 files)

###!!! ASSERTION: Form control has a frame, but it's not a form frame: 'Error',
file nsGenericHTMLElement.cpp, line 2250

http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/content/html/content/src/nsGenericHTMLElement.cpp&rev=1.566&root=/cvsroot&mark=2250#2224

STEPS TO REPRODUCE:
1. load testcase
2. click on <select>
3. move mouse over dropdown menu

It only occurs if there is an <optgroup> present
Attached file Testcase
the problem is that nsHTMLOptGroupElement::HandleDOMEvent calls GetFormControlFrame.

It shouldn't be doing that.  Its frame will NEVER be a form control frame --
it'll be a block.  So it should just call GetPrimaryFrame(PR_FALSE).

Mats, want to fix this?  If not, let me know and I will...
Assignee: nobody → mats.palmgren
Attached file Testcase #2
Attached patch Patch rev. 1Splinter Review
Attachment #173441 - Flags: superreview?(bzbarsky)
Attachment #173441 - Flags: review?(bzbarsky)
Comment on attachment 173441 [details] [diff] [review]
Patch rev. 1

r+sr=bzbarsky
Attachment #173441 - Flags: superreview?(bzbarsky)
Attachment #173441 - Flags: superreview+
Attachment #173441 - Flags: review?(bzbarsky)
Attachment #173441 - Flags: review+
Checked in 2005-02-04 20:13 PDT

-> FIXED
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: