Commit df2908f3 authored by Ian Kilpatrick's avatar Ian Kilpatrick Committed by Commit Bot

[cleanup] Remove some dead table pagination code.

There should be no behaviour change.

Change-Id: I95e258c66b2d79b8cb172fbae97f6716c2cf66c0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1736438
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Auto-Submit: Ian Kilpatrick <ikilpatrick@chromium.org>
Reviewed-by: default avatarMorten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#684084}
parent b97951a5
......@@ -116,12 +116,4 @@ LayoutState::~LayoutState() {
}
}
LayoutUnit LayoutState::PageLogicalOffset(
const LayoutBox& child,
const LayoutUnit& child_logical_offset) const {
if (child.IsHorizontalWritingMode())
return pagination_offset_.Height() + child_logical_offset;
return pagination_offset_.Width() + child_logical_offset;
}
} // namespace blink
......@@ -76,12 +76,6 @@ class LayoutState {
bool IsPaginated() const { return is_paginated_; }
// The page logical offset is the object's offset from the top of the page in
// the page progression direction (so an x-offset in vertical text and a
// y-offset for horizontal text).
LayoutUnit PageLogicalOffset(const LayoutBox&,
const LayoutUnit& child_logical_offset) const;
LayoutUnit HeightOffsetForTableHeaders() const {
return height_offset_for_table_headers_;
}
......
......@@ -849,11 +849,6 @@ void LayoutTable::UpdateLayout() {
ComputeLayoutOverflow(ClientLogicalBottom());
UpdateAfterLayout();
if (state.IsPaginated() && IsPageLogicalHeightKnown()) {
block_offset_to_first_repeatable_header_ = state.PageLogicalOffset(
*this, top_section ? top_section->LogicalTop() : LayoutUnit());
}
}
// FIXME: This value isn't the intrinsic content logical height, but we need
......
......@@ -398,13 +398,6 @@ class CORE_EXPORT LayoutTable final : public LayoutBlock {
bool IsLogicalWidthAuto() const;
// When table headers are repeated, we need to know the offset from the block
// start of the fragmentation context to the first occurrence of the table
// header.
LayoutUnit BlockOffsetToFirstRepeatableHeader() const {
return block_offset_to_first_repeatable_header_;
}
const char* GetName() const override { return "LayoutTable"; }
// Whether a table has opaque foreground depends on many factors, e.g. border
......@@ -603,7 +596,6 @@ class CORE_EXPORT LayoutTable final : public LayoutBlock {
mutable unsigned collapsed_outer_border_start_overflow_;
mutable unsigned collapsed_outer_border_end_overflow_;
LayoutUnit block_offset_to_first_repeatable_header_;
LayoutUnit row_offset_from_repeating_header_;
LayoutUnit row_offset_from_repeating_footer_;
LayoutUnit old_available_logical_height_;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment