558574 - tfoot placement wrong (print preview, print)

Status: RESOLVED FIXED

(Core :: Layout: Tables, defect)

Description

[Bernd]

Attachment: reftest

see attached test case, the expected rendering is in the reference

Comment 1

[Bernd]

Attachment: reference

compare the reftest in IE print preview how it should be rendered

Comment 2

[Bernd]

Attachment: test for another failure

look in print preview,  this executes the code 

http://mxr.mozilla.org/mozilla-central/source/layout/tables/nsTableFrame.cpp#2765
where roc's comment is exactly to the point.

Comment 3

[Bernd]

Attachment: testcase for the pushchildren above

it also fails.

Comment 4

[Bernd]

Attachment: test case for problem on first row

Comment 5

[Bernd]

Attachment: nested page breaks also fail

Comment 6

[Bernd]

Attachment: split code not executed if all content fits on one page

Comment 7

[Bernd]

Attachment: patch

Comment 8

[Bernd]

Comment on attachment 443037 [details] [diff] [review]
patch

Robert this are the bugs that I fixed while writing some reftests for table pagination.

I pushed this already over the try server

Comment 9

[Bernd]

The tests are in bug 558575

Comment 10

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 443037 [details] [diff] [review]
patch

+  PRBool InternalBreakBefore() const;
+  PRBool InternalBreakAfter() const;

Make these "HasInternalBreak*" so they don't look like verbs.

Comment 11

[Bernd]

http://hg.mozilla.org/mozilla-central/rev/fcff7673a6ae