125543 - Prints page 1 and 3 but not 2...

Status: VERIFIED FIXED

(Core :: Printing: Output, defect, P1)

Description

[Matthew Ohlmeyer]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:0.9.8) Gecko/20020204
BuildID:    2002020406

I printed this webpage:
http://www.snopes2.com/cokelore/acid.htm
And the footer said page 1 of 2...  Page 1 had the stuff I would expect for page
1.  Page 2 had the stuff I would've expected for a page 3, if there had been
one.  And all the content that should've been on page 2, was skipped all together.


Reproducible: Always
Steps to Reproduce:
1. Open web page: http://www.snopes2.com/cokelore/acid.htm
2. Click the prtint button.
3. Click Print.

Actual Results:  Two pages are printed, a large portion of the webpage is left
out between page one and two.

Expected Results:  Three pages should be printed, with all of the origional
webpage content shown.

Printer is a Lexmak 3200, using default WinXP driver.

Comment 1

[sujay]

confirming. I can reproduce this on my win 98 system with 2/14 trunk build.
2nd page does not get printed out. its skipped.

Comment 2

[John Levon]

confirmed in print preview, linux as of cvs yesterday. setting to OS->all.

There are a few print missing pages bugs, not sure if this is a dupe of
one of them...

Comment 3

[Andrew Hagen]

Confirming on basis of comments.

Comment 4

[rods (gone)]

lost content for printing

Comment 5

[karnaze (gone)]

Attachment: somewhat reduced test case

Comment 6

[Kevin McCluskey (gone)]

nsbeta1+

Comment 7

[karnaze (gone)]

Attachment: somewhat reduced test case

Comment 8

[karnaze (gone)]

Attachment: patch to fix the bug

The patch corrects some flaws in the special height reflow mechanism. Row
groups are prevented from splitting in the reflow preceeding the special height
reflow. Cells are only notified that they should observe a percent height
element if the element is inside the table's cell. Percent height elements
inside the body will have a height based on the page height when printing. The
risk of the patch is limited to pages which have percent height table related
elements or percent height elements inside table cells.

Comment 9

[Alexandru Savulov]

Comment on attachment 78356 [details] [diff] [review]
patch to fix the bug

r= alexsavulov

Comment 10

[Alexandru Savulov]

Comment on attachment 78356 [details] [diff] [review]
patch to fix the bug

r= alexsavulov

Comment 11

[Alexandru Savulov]

double r= means 1/2sr= ;-)

Comment 12

[kinmoz]

Comment on attachment 78356 [details] [diff] [review]
patch to fix the bug

sr=[e-mail]

Comment 13

[karnaze (gone)]

The patch has been checked into the trunk. I'm leaving it open because it may 
make it into the m1.0 branch.

Comment 14

[Jaime Rodriguez, Jr.]

Remove adt1.0.0. Let it bake on the trunk until Monday, let Sujay take a good
look at it, then renominate for inclusion in the 1.0 branch.

Comment 15

[Asa Dotzler [:asa]]

Comment on attachment 78356 [details] [diff] [review]
patch to fix the bug

a=asa (on behalf of drivers) for checkin to the 1.0 branch

Comment 16

[Kevin McCluskey (gone)]

sujay: Could you verify the fix on the trunk? Thanks!

Comment 17

[Amarendra Hanumanula]

 Printing all the 3 pages correctly. Verified on 2002-04-12-06-trunk. WIN2K.

Comment 18

[Kevin McCluskey (gone)]

Resolving as FIXED. Please add fixed1.0.0 keyword after the fix has been checked
into the 1.0.0 branch.

Comment 19

[Jaime Rodriguez, Jr.]

This is a pretty big patch.  It looks like amar tested the case presented in
this bug, but what kind of testing needs to be done around the changes in the
code.  For example, it looks like there were a lot of changes to tables.

Comment 20

[karnaze (gone)]

The patch can affect pages with heights on table related elements as well as 
percent heights on elements inside table cells and the body. There are a number 
of regression tests that test these things which have passed. The patch has been 
baking on the trunk for 1 week and I haven't seen any regressions as a result of 
this patch. I hope amar and sujay have been looking for regressions as well.

Comment 21

[sujay]

verified on 4/16 trunk build.

Comment 22

[Paul Wyskoczka]

adt1.0.0+ on behalf of the adt.  Please check into the branch today and add
fixed1.0.0 to the keyword field.

Comment 23

[sujay]

verified in 4/17 branch build. adding "verified1.0.0" keyword.