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

Issue 621359 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Use other robhogan account instead.
Closed: Dec 2016
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

Handle layout of multiple theads in a table on printed page

Project Member Reported by robhogan@chromium.org, Jun 19 2016

Issue description

Fix the display glitch in this test case.


 
single-line-cells-many-repeating-theads.html
2.0 KB View Download
Owner: robhogan@chromium.org
Status: Assigned (was: Unconfirmed)
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5aca76371f4c851adb4b1ac398ac45d62073967b

commit 5aca76371f4c851adb4b1ac398ac45d62073967b
Author: robhogan <robhogan@gmail.com>
Date: Tue Jun 21 10:26:24 2016

Handle pagination breaking on rows correctly when a table has >1 section

When a table has more than one section, layoutRows() needs to have an
up-to-date logicalTop() for the section. Otherwise it will calculate
the wrong offset from the top of the page for the row when figuring out whether
to put a strut on it.

BUG= 621359 

Review-Url: https://codereview.chromium.org/2076413002
Cr-Commit-Position: refs/heads/master@{#400941}

[add] https://crrev.com/5aca76371f4c851adb4b1ac398ac45d62073967b/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-in-multiple-table-sections-expected.txt
[add] https://crrev.com/5aca76371f4c851adb4b1ac398ac45d62073967b/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-in-multiple-table-sections.html
[modify] https://crrev.com/5aca76371f4c851adb4b1ac398ac45d62073967b/third_party/WebKit/Source/core/layout/LayoutTable.cpp

Status: Fixed (was: Assigned)

Sign in to add a comment