362084 - On print preview: ASSERTION: data loss - incomplete row needed more height than available, on top of page

Status: RESOLVED WORKSFORME

(Core :: Layout: Tables, defect)

Description

[Martijn Wargers (dead)]

See upcoming testcase, which triggers this kind of assertion on print preview.
###!!! ASSERTION: data loss - incomplete row needed more height than available,
on top of page: 'rowMetrics.height <= rowReflowState.availableHeight', file c:/m
ozilla/mozilla/layout/tables/nsTableRowGroupFrame.cpp, line 1134

This is a testcase that is more or less derived from the testcase from bug 359132, but this one is slightly more minimised and triggers this different assertion when opening print preview.

Comment 1

[Martijn Wargers (dead)]

Attachment: testcase

Comment 2

[Bernd]

does that also assert on the reflow branch?

Comment 3

[Martijn Wargers (dead)]

(In reply to comment #2)
> does that also assert on the reflow branch?

Yes.

Comment 4

[Bernd]

<insert your local fword here>  no cheap bug fixes :-(, does the assert also appear in bug 359132 ?

Comment 5

[Martijn Wargers (dead)]

(In reply to comment #4)
> <insert your local fword here>  no cheap bug fixes :-(, does the assert also
> appear in bug 359132 ?

Heh, my current debug reflow branch build crashes when closing print preview, but I get the usual round of assertions on print preview, so I bet that bug is also there on the reflow branch.

Comment 6

[Martijn Wargers (dead)]

I'm still seeing this assertion when in landscape mode.

Comment 7

[Bernd]

me too :-(

Comment 8

[Martijn Wargers (dead)]

The attached testcase doesn't seem to assert anymore on print preview, however the same assertion can be seen with the testcase from bug 424291: https://bugzilla.mozilla.org/attachment.cgi?id=310925

Comment 9

[Bernd]

this is wfm also comment 8

Comment 10

[Martijn Wargers (dead)]

The first testcase in bug 470495 triggers this assertion still (among others).