"ASSERTION: Item was successful, but node from collection was null!" involving DOMNodeRemoved event

RESOLVED FIXED

Status

()

RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: jruderman, Assigned: smaug)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
assertion, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

12 years ago
Created attachment 279399 [details]
testcase (from bug 339501)

Loading the testcase triggers:

###!!! ASSERTION: Item was successful, but node from collection was null!: 'node', file /Users/jruderman/trunk/mozilla/layout/forms/nsListControlFrame.cpp, line 1218

###!!! ASSERTION: could not get option element by index!: 'optionElement != nsnull', file /Users/jruderman/trunk/mozilla/layout/forms/nsListControlFrame.cpp, line 1177
Assignee: nobody → Olli.Pettay
Created attachment 279424 [details] [diff] [review]
when rebuilding options array, update the selectedIndex too

RebuildOptionsArray() is called only when something "unusual" has happened, like
mutation event listener has modified options list.
Feel free to approval1.9+ too.
Attachment #279424 - Flags: superreview?(jonas)
Attachment #279424 - Flags: review?(jonas)
Comment on attachment 279424 [details] [diff] [review]
when rebuilding options array, update the selectedIndex too

Though it would be great to get a mochitest too.
Attachment #279424 - Flags: superreview?(jonas)
Attachment #279424 - Flags: superreview+
Attachment #279424 - Flags: review?(jonas)
Attachment #279424 - Flags: review+
Attachment #279424 - Flags: approval1.9+
I'll check in a mochitest too.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
OS: Mac OS X → All
Hardware: PC → All
You need to log in before you can comment on or make changes to this bug.