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

[LayoutNG] Check if a fragment should paint based on VisualOverflowRect.

... instead of VisualSelfOverflowRect. Children wouldn't have painted
if enabled with FragmentItems, or BoxPainting enabled.

Originally added in:
https://chromium-review.googlesource.com/c/chromium/src/+/1628634

Bug: 988015
Change-Id: I4925f51b66f67f74504d2a4e1fb25b157aa060ac
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1859681Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#705666}
parent cc16d953
......@@ -1175,8 +1175,7 @@ bool NGBoxFragmentPainter::ShouldPaint(
const NGPhysicalBoxFragment& fragment = PhysicalFragment();
if (!fragment.IsInlineBox()) {
return paint_state.LocalRectIntersectsCullRect(
ToLayoutBox(fragment.GetLayoutObject())
->PhysicalSelfVisualOverflowRect());
ToLayoutBox(fragment.GetLayoutObject())->PhysicalVisualOverflowRect());
}
NOTREACHED();
return false;
......
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