Created attachment 299480 [details] testcase Loading the testcase triggers: ###!!! ASSERTION: row count did not change by the amount suggested, check caller: 'rowCount == mRowCount', file /Users/jruderman/trunk/mozilla/layout/xul/base/src/tree/src/nsTreeBodyFrame.cpp, line 1817 WARNING: row count changed unexpectedly: 'mRowCount == rowCount', file /Users/jruderman/trunk/mozilla/layout/xul/base/src/tree/src/nsTreeBodyFrame.cpp, line 2764 ###!!! ASSERTION: bad index: 'aIndex >= 0 && aIndex < mRows.Count()', file /Users/jruderman/trunk/mozilla/layout/xul/base/src/tree/src/nsTreeContentView.cpp, line 284 The "bad index" assertion is covered by bug 366203, but the "row count did not change by the amount suggested" assertion doesn't seem to be covered by other bug reports.
Looks to me that ContentInserted needs to call InsertRowFor to deal with <option> correctly (it already does that for treeitem/treeseparator/optgroup).
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: xptoolkit.trees → xptoolkit.widgets
Created attachment 396612 [details] crash stack The testcase crashes on trunk [@ nsTreeContentView::InsertRow].
Like bug 479931.
This looks like a straight duplicate to me; the only difference I noticed is that there's a slightly different non-tree node in the tree. The crash would probably occur with a text node if you cared to give it a try.
Looks like a duplicate to me too. The patch in bug 479931 fixes this.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 479931
You need to log in before you can comment on or make changes to this bug.