Commit e6224f32 authored by Vladimir Levin's avatar Vladimir Levin Committed by Commit Bot

Ensure to check for paint invalidation on tablerow vis rect changes.

When the visual rect of the table row changes due to the cell's frame
rect changing, we don't current invalidate the visual rect.

This patch addresses that.

R=wangxianzhu@chromium.org

Bug: 892647
Change-Id: Iaf5de6f4b801175bb6e0a4e3e6df92468400451e
Reviewed-on: https://chromium-review.googlesource.com/c/1265896Reviewed-by: default avatarXianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: vmpstr <vmpstr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597349}
parent ed923038
...@@ -288,10 +288,13 @@ LayoutTableRow* LayoutTableRow::CreateAnonymousWithParent( ...@@ -288,10 +288,13 @@ LayoutTableRow* LayoutTableRow::CreateAnonymousWithParent(
} }
void LayoutTableRow::ComputeOverflow() { void LayoutTableRow::ComputeOverflow() {
const auto& old_visual_rect = SelfVisualOverflowRect();
ClearAllOverflows(); ClearAllOverflows();
AddVisualEffectOverflow(); AddVisualEffectOverflow();
for (LayoutTableCell* cell = FirstCell(); cell; cell = cell->NextCell()) for (LayoutTableCell* cell = FirstCell(); cell; cell = cell->NextCell())
AddOverflowFromCell(cell); AddOverflowFromCell(cell);
if (old_visual_rect != SelfVisualOverflowRect())
SetShouldCheckForPaintInvalidation();
} }
void LayoutTableRow::AddOverflowFromCell(const LayoutTableCell* cell) { void LayoutTableRow::AddOverflowFromCell(const LayoutTableCell* cell) {
......
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