Jesse Ruderman Reporter
	Description • 17 years ago

###!!! ASSERTION: assigned width smaller than min: 'col_width >= colFrame->GetMinCoord()', file /Users/jruderman/trunk/mozilla/layout/tables/BasicTableLayoutStrategy.cpp, line 960

###!!! ASSERTION: didn't subtract all that we added: 'space == 0 && ((l2t == FLEX_PCT_LARGE) ? (-0.001f < basis.f && basis.f < 0.001f) : (basis.c == 0))', file /Users/jruderman/trunk/mozilla/layout/tables/BasicTableLayoutStrategy.cpp, line 972

###!!! ASSERTION: bad width: 'Not Reached', file /Users/jruderman/trunk/mozilla/layout/generic/nsLineLayout.cpp, line 180

###!!! ASSERTION: Computed overflow area must contain frame bounds: 'aNewSize.width == 0 || aNewSize.height == 0 || aOverflowArea->Contains(nsRect(nsPoint(0, 0), aNewSize))', file /Users/jruderman/trunk/mozilla/layout/generic/nsFrame.cpp, line 5317

	Daniel Holbert [:dholbert] Assignee
	Comment 1 • 17 years ago

Simplified the test case a bit.

Problem occurs for values of padding >= 50%.  

I'm not totally familiar with padding, but it seems like the problem happens because we've got two divs in the same row that are each trying to take up over half of the row.

	Jesse Ruderman Reporter
	Comment 2 • 17 years ago

Hmm, the new testcase triggers width-related assertions but doesn't make scrollbars appear.

	Daniel Holbert [:dholbert] Assignee
	Comment 3 • 17 years ago

oops, Jesse's right -- my second testcase only triggers the first two assertions.  

Maybe that will make that section of the problem easier to debug, in any case.

	Daniel Holbert [:dholbert] Assignee
	Comment 4 • 17 years ago

Filed a separate bug for my reduced testcase, which triggers the first 2 assertions only: bug 385533

	Daniel Holbert [:dholbert] Assignee
	Comment 5 • 17 years ago

My first patch for bug 385533 (attachment 269715 [details] [diff] [review]) fixes this bug's testcases.  (Just posted the patch, so it hasn't yet been reviewed)

	Daniel Holbert [:dholbert] Assignee
	Comment 6 • 17 years ago

I posted an nscoord_MAX-checking patch on bug 367673, and it covers this bug as
well.

	Daniel Holbert [:dholbert] Assignee
	Comment 7 • 17 years ago

Fixed by checkin for bug 367673.

	Jesse Ruderman Reporter
	Comment 8 • 17 years ago

Crashtests checked in.  They don't check for huge widths/heights, but they do check for the resulting assertions.