Closed
Bug 1144641
Opened 11 years ago
Closed 8 years ago
crash in nsTableFrame::FixupPositionedTableParts with vertical writing mode
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: jfkthame, Unassigned)
References
Details
(Keywords: testcase)
Attachments
(1 file)
|
1.81 KB,
text/html
|
Details |
Running the testcase from gfx/tests/crashtests/783041-2.html with writing-mode:vertical-lr applied to the <body> element will generate a number of assertions, then crash in nsTableFrame::FixupPositionedTableParts.
Although we don't yet implement vertical layout for table elements (and therefore some assertions here are not surprising), we should at least avoid crashing.
|
Reporter | |
Comment 1•11 years ago
|
||
Testcase created by adding vertical writing mode to the original gfx crashtest.
NB: crashes a current trunk build if writing-mode support is enabled.
|
|
Reporter | |
Comment 2•11 years ago
|
||
Assertions we hit when loading this testcase include:
0:26.47 [32686] ###!!! ASSERTION: Table width is less than the sum of its columns' min widths: '!(aWidthType == BTLS_FINAL_WIDTH && aWidth < guess_min)', file layout/tables/BasicTableLayoutStrategy.cpp, line 816
0:26.63 [32686] ###!!! ASSERTION: didn't subtract all that we added: '(space == 0 || space == nscoord_MAX) && ((l2t == FLEX_PCT_LARGE) ? (-0.001f < basis.f && basis.f < 0.001f) : (basis.c == 0 || basis.c == nscoord_MAX))', file layout/tables/BasicTableLayoutStrategy.cpp, line 1061
0:26.78 [32686] ###!!! ASSERTION: Shouldn't be incomplete if availableBSize is UNCONSTRAINED.: 'aReflowState.AvailableBSize() != NS_UNCONSTRAINEDSIZE', file layout/generic/nsBlockFrame.cpp, line 1570
0:26.87 [32686] ###!!! ASSERTION: frame tree not empty, but caller reported complete status: 'aSubtreeRoot->GetPrevInFlow()', file layout/base/nsLayoutUtils.cpp, line 6876
0:27.56 [32686] ###!!! ASSERTION: frame tree not empty, but caller reported complete status: 'start == end || IsInLetterFrame(aSubtreeRoot)', file layout/base/nsLayoutUtils.cpp, line 6890
0:26.47 [32686] ###!!! ASSERTION: Table width is less than the sum of its columns' min widths: '!(aWidthType == BTLS_FINAL_WIDTH && aWidth < guess_min)', file layout/tables/BasicTableLayoutStrategy.cpp, line 816
0:26.63 [32686] ###!!! ASSERTION: didn't subtract all that we added: '(space == 0 || space == nscoord_MAX) && ((l2t == FLEX_PCT_LARGE) ? (-0.001f < basis.f && basis.f < 0.001f) : (basis.c == 0 || basis.c == nscoord_MAX))', file layout/tables/BasicTableLayoutStrategy.cpp, line 1061
Pushed by mpalmgren@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e3f73fdc82af
Add a crashtest.
|
||
Comment 6•8 years ago
|
||
The test was green on Try so I pushed it and calling this WFM.
|
||
Comment 7•8 years ago
|
||
| bugherder | ||
You need to log in
before you can comment on or make changes to this bug.
Description
•