272830 - to small row group is not pushed on next page

Status: RESOLVED FIXED

(Core :: Printing: Output, defect)

Description

[Bernd]

load testcase in print preview see that the second page is empty one should be
able to read last line. This is probably a dupe but I can't find it. I will
attach soon.

Comment 1

[Bernd]

Attachment: testacse

Comment 2

[Bernd]

Attachment: patch

Comment 3

[Bernd]

Attachment: revised patch

Comment 4

[Boris Zbarsky [:bzbarsky]]

It'll take me a bit to get to this -- my net connection is spotty.... :(

Comment 5

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 167924 [details] [diff] [review]
revised patch

>Index: nsTableFrame.cpp

>+        aDesiredSize.height = borderPadding.bottom + GetCellSpacingY() +
>+                              lastChildReflowed->GetRect().YMost();

Why is this still adding in the cellspacing?  I don't think it should...

r+sr=bzbarsky with that fixed or a comment explaining why it's correct.

Comment 6

[Bernd]

the row groups are inset at the top by a border padding and a cellspacing this
is reflected in the YMost. But a table has the borderpadding bottom and  another
cellspacing below the table and this, as the cellspacing above the row group not
included (see CSS 2.1 "Next is the layer containing the row groups. Each row
group extends from the top left corner of its topmost cell in the first column
to the bottom right corner of its bottommost cell in the last column."). So I
believe the patch is correct.

Comment 7

[Boris Zbarsky [:bzbarsky]]

Oh, if we're putting cellspacing below the last lowgroup then the patch is in
fact correct...

Comment 8

[Bernd]

fix checked in

Comment 9

[Jeff Walden [:Waldo]]

http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/layout/reftests/printing/reftest.list&rev=1.1