New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 652344 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Use other robhogan account instead.
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Page break in one table causes headers for all tables on the page to repeat

Reported by mitch.ga...@gofmx.com, Oct 3 2016

Issue description

UserAgent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2785.116 Safari/537.36

Example URL:
https://jsfiddle.net/9ao5v7h5/

Steps to reproduce the problem:
1. Cause a table to break across pages
2. Add a second table after the page-broken one
3. Observe the repeated header of the page-broken table also contains the header of the second table

What is the expected behavior?
Only the header for the table that's broken across pages is repeated.

What went wrong?
Headers are repeated for all tables on the page, not just the table that was broken across pages.

Does it occur on multiple sites: Yes

Is it a problem with a plugin? No 

Did this work before? N/A 

Does this work in other browsers? Yes 

Chrome version: 53.0.2785.116  Channel: stable
OS Version: 6.1 (Windows 7, Windows Server 2008 R2)
Flash Version: Shockwave Flash 23.0 r0
 
Labels: Needs-Bisect
Cc: msten...@opera.com
Components: -Blink Blink>Layout
Please find the attached chrome and Firefox screenshots, as we observed the same behaviour in both the browsers (Chrome-53.0.2785.143, Firefox). Could you please elaborate the expected behaviour for the same to reproduce the issue.
652344-Chrome.PNG
142 KB View Download
652344-Firefox.PNG
104 KB View Download

Comment 4 Deleted

Labels: Needs-Bisect
Sorry, should have indicated that the page breaking behavior occurs when printing.  To reproduce, click the "Print" button in the JSFiddle (https://jsfiddle.net/9ao5v7h5/) and then observe the table header on the second page of the output.

The expected behavior is demonstrated in the Firefox and IE 11 outputs (see attached), where only the table that's broken across pages repeats its header.  Chrome's behavior (see attached) is to repeat the headers of all tables on the page, in the same visual space, even tables that aren't broken across pages.
Chrome.png
92.7 KB View Download
Firefox.png
97.5 KB View Download
IE 11.png
86.3 KB View Download
Chrome.pdf
57.7 KB Download
Firefox.pdf
58.6 KB Download
IE 11.pdf
31.3 KB Download
Labels: -Type-Bug -Needs-Feedback -Needs-Bisect hasbisect-per-revision M-54 OS-Linux OS-Mac Type-Bug-Regression
Owner: robhogan@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on windows mac and Linux using Chrome stable (53.0.2785.143),beta (54.0.2840.50) .

But this is working fine on Dev(55.0.2873.0) and canary(55.0.2882.0). Seems the issue got fixed on later versions. Worked on reverse bisect and below is the info

Manual Bisect:
------------
first Good build:55.0.2862.0
last Bad build:55.0.2861.0

Bisect Tool Info:
----------------
 https://chromium.googlesource.com/chromium/src/+log/5dc5867ab6200abb2f02392a16267ecec62d676e..8bf47bb4e1d0ea9599bc443906fc26c3ce86f152


suspect:
-------
https://chromium.googlesource.com/chromium/src/+/8bf47bb4e1d0ea9599bc443906fc26c3ce86f152

robhogan@ Please reassign if this is not related to your change.

Comment 8 by robho...@gmail.com, Oct 6 2016

Labels: -hasbisect-per-revision Merge-Request-54
Yup, this is fixed in both unstable branches.
Any idea on when this patch will be release to Google Chrome? I am on "Version 53.0.2785.143 m" and still see this issue.

We are having a lot of clients complaining about this issue and are having them switch to IE to get around this issue for the time being.
Labels: Merge-Request-53

Comment 11 by dimu@chromium.org, Oct 7 2016

Labels: -Merge-Request-53 Merge-Review-53 Hotlist-Merge-Review
[Automated comment] Request affecting a post-stable build (M53), manual review required.

Comment 12 by dimu@chromium.org, Oct 7 2016

Labels: -Merge-Request-54 Merge-Review-54
[Automated comment] Less than 2 weeks to go before stable on M54, manual review required.

Comment 13 by dimu@chromium.org, Oct 7 2016

Labels: -Merge-Request-53 Merge-Review-53 Hotlist-Merge-Review
[Automated comment] Request affecting a post-stable build (M53), manual review required.

Comment 14 Deleted

We don't have any M53 Stable release plan for Chrome Desktop.
Cc: bustamante@chromium.org
+ bustamante@ for M54 Merge Review
Labels: -Merge-Review-54 Merge-Approved-54
Approving for M54, this is pretty visible when it happens, and the CL looks safe.
Looks like Team Chrome has made some progress. When can we expect this fix pushed out in Production? The situation is pretty bad with the prints right now. 
Labels: ReleaseBlock-Stable
Thanks for the fix, verified in latest canary- 56.0.2886.0 , works as expected.

IMPORTANT: M54 will be promoted to Stable soon! Please make sure to merge the fix before 4:00 PM PT today, Monday- 10/10 if you want to make the fix to STABLE channel.

Comment 20 by robho...@gmail.com, Oct 10 2016

Status: Fixed (was: Assigned)
Landed to M54 with https://codereview.chromium.org/2400303004
Labels: -Merge-Approved-54 merge-merged-2840
Thanks for the merge. 

Confirming the fix landed in M54 - https://chromium.googlesource.com/chromium/src.git/+/ee7ca0fdc96ab4ba61d0be876c8e779adced302d

Updating labels accordingly.
@robhogan@chromium.org

When will this fix be available in my browser?

Sign in to add a comment