1155230 - Printing nested table with empty row causes unexpected results

Status: VERIFIED FIXED

(Core :: Layout: Tables, defect)

Description

[David St.Clair]

Attachment: bug.html

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.90 Safari/537.36

Steps to reproduce:

Open attached file bug.html in Firefox 37.0.1 (Windows 7). Click the print button. On printout, the words "This text should print on the second page." and a button that says "print" should be on the second page.

This works properly in Firefox 36.0.4, Chrome 42.0.2311.90 (64 bit), and IE 11.0.9600.17691.


Actual results:

3 pages are printed. The first 2 are blank and the last one has the "print" button on it.


Expected results:

2 pages should have been printed with the second page containing the words "This text should print on the second page." and a button that says "Print".

Comment 1

[Alice0775 White]

Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=16ea12664917&tochange=c2184fba8cc1

Regressed by: Bug 1108104

Comment 2

[Mats Palmgren (inactive)]

Fwiw, I think you can workaround this bug by adding a <td> in the empty <tr>.

Comment 3

[Mats Palmgren (inactive)]

If I remove the "incremental reflow hack" in nsTableOuterFrame::Reflow that
was moved from nsSimplePageSequenceFrame::Reflow in bug 1108104, then the
bug does not occur.  (I don't think moving it back is an option though.)
The problem occurs because the inner table frame has an OverflowList when
we do that early return.

Comment 4

[Lawrence Mandel [:lmandel] (use needinfo)]

Tracking regression for 38+. This is not significant enough to warrant a point release for 37.

Comment 5

[Mats Palmgren (inactive)]

Bug 1152354 fixed this I think.

Comment 6

[Sylvestre Ledru [:Sylvestre]]

Udpating the tracking flags then

Comment 7

[Sayed Mohammad Amir [:smamir]]

Reproduced with Firefox Nightly 40.0a1 with the instructions from comment 0 on Windows 7 64 Bit.
Build ID: 20150416030209
User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:40.0) Gecko/20100101 Firefox/40.0

Verified as fixed with:
Firefox 38.0.1
Build ID: 20150513174244
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Firefox/38.0

Firefox Beta 39.0
Build ID: 20150523155636
User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:39.0) Gecko/20100101 Firefox/39.0

Firefox Aurora 40.0a2
Build ID: 20150526004004
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:40.0) Gecko/20100101 Firefox/40.0

[bugday-20150527]

Comment 8

[Orvi]

Test it on  linux 64 Bit

verified as fixed with:
Firefox  Beta 39.0
Name 	        Iceweasel
Version 	39.0
Build ID 	20150527225439
User Agent 	Mozilla/5.0 (X11; Linux x86_64; rv:39.0) Gecko/20100101 Firefox/39.0 Iceweasel/39.0

Firefox Aurora 40.0a2
Name 	        Iceweasel
Version 	40.0a2
Build ID 	20150527004004
User Agent 	Mozilla/5.0 (X11; Linux x86_64; rv:40.0) Gecko/20100101 Firefox/40.0 Iceweasel/40.0a2

Firefox released version 38.0.1 
Name            Iceweasel
Version 	38.0.1
Build ID 	20150525043631
User Agent 	Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Firefox/38.0 Iceweasel/38.0.1

[bugday-20150603]

Comment 9

[Ashickur Rahman]

I have successfully reproduced the bug in nightly firefox-40.0a1. 

The bug is fixed on latest nightly 41.0a1 (User Agent 	Mozilla/5.0 (X11; Linux x86_64; rv:41.0) Gecko/20100101 Firefox/41.0 , build ID	20150609081916).