Closed Bug 382746 Opened 17 years ago Closed 17 years ago

"ASSERTION: Lists not the same length" with XUL grid

Categories

(Core :: XUL, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jruderman, Assigned: neil)

References

Details

(Keywords: assertion, testcase)

Attachments

(2 files)

Attached file testcase (deleted) —
###!!! ASSERTION: Lists not the same length.: 'Not Reached', file /Users/jruderman/trunk/mozilla/layout/xul/base/src/nsSprocketLayout.cpp, line 347
Flags: blocking1.9?
// If we encounter a completely bogus box size, we just leave this child completely // alone and continue through the loop to the next child. if (childBoxSize->bogus) { childComputedBoxSize = childComputedBoxSize->next; childBoxSize = childBoxSize->next; count++; x = nextX; y = nextY; continue; } Except we don't actually continue to the next child...
Attached patch Proposed patch (deleted) — Splinter Review
As far as I can tell from the grid code, a bogus box size still has a child.
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #267002 - Flags: superreview?(bzbarsky)
Attachment #267002 - Flags: review?(enndeakin)
Attachment #267002 - Flags: review?(enndeakin) → review+
Comment on attachment 267002 [details] [diff] [review] Proposed patch Sure.
Attachment #267002 - Flags: superreview?(bzbarsky) → superreview+
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
Depends on: 383274
Flags: blocking1.9?
Depends on: 383236
Crashtest checked in.
Flags: in-testsuite? → in-testsuite+
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: xptoolkit.xul → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: