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

RESOLVED WORKSFORME

Status

()

Core
Layout: Tables
RESOLVED WORKSFORME
12 years ago
8 years ago

People

(Reporter: Martijn Wargers (zombie), Unassigned)

Tracking

({assertion, testcase})

Trunk
x86
Windows XP
assertion, testcase
Points:
---
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
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.
(Reporter)

Comment 1

12 years ago
Created attachment 246783 [details]
testcase

Comment 2

12 years ago
does that also assert on the reflow branch?
(Reporter)

Comment 3

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

Yes.

Comment 4

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

Comment 5

12 years ago
(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.
(Reporter)

Comment 6

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

Comment 7

11 years ago
me too :-(
(Reporter)

Comment 8

10 years ago
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

10 years ago
this is wfm also comment 8
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → WORKSFORME

Updated

10 years ago
Flags: in-testsuite?
(Reporter)

Comment 10

8 years ago
The first testcase in bug 470495 triggers this assertion still (among others).
You need to log in before you can comment on or make changes to this bug.