Created attachment 360991 [details] testcase WARNING: Someone passed native anonymous content directly into frame construction. Stop doing that!: file /Users/jruderman/central/layout/base/nsCSSFrameConstructor.cpp, line 7924 ###!!! ASSERTION: out of bounds: 'PRInt32(aIndex) >= 0 && aIndex <= l', file /Users/jruderman/central/layout/base/nsChildIterator.h, line 132 ###!!! ASSERTION: anonymous nodes should not be in child lists: '!aOldChild->IsRootOfAnonymousSubtree()', file /Users/jruderman/central/layout/base/nsCSSFrameConstructor.cpp, line 12404
Created attachment 361273 [details] [diff] [review] Stupid first-letter case... I hate editor. :(
10 years ago
Summary: "ASSERTION: out of bounds" with contentEditable, table, overflow:scroll → [FIX]"ASSERTION: out of bounds" with contentEditable, table, overflow:scroll
Am I correct that all you're changing here is removing the assertion and swapping the order of the then/else?
Yeah, basically. I'm just avoiding making any seek() calls in the non-XBL case when the index is -1.
Comment on attachment 361273 [details] [diff] [review] Stupid first-letter case... Ah, the else is now an else if; that's the part I missed.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.