Commit 362a38e0 authored by Yoshifumi Inoue's avatar Yoshifumi Inoue Committed by Commit Bot

Make NGBoxFragmentPainter::PaintTextItem() to paint text if IsVisibleToPaint()

This patch changes |NGBoxFragmentPainter::PaintTextItem()| to paint text if
|IsVisibleToPaint()|. This is follow-up of the patch[1] uses opposite condition.


[1] http://crrev.com/c/1892981 [FragmentItem] Fix DCHECK in PaintTextItem

Bug: 982194
Change-Id: Id2e229169fabae9c00c42197e0fab9a4b9382c81
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892982
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: default avatarIan Kilpatrick <ikilpatrick@chromium.org>
Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Auto-Submit: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#712004}
parent ea894cbe
......@@ -1180,7 +1180,7 @@ void NGBoxFragmentPainter::PaintTextItem(const NGInlineCursor& cursor,
// Need to check the style of each text items because they can have different
// styles than its siblings if inline boxes are culled.
if (UNLIKELY(IsVisibleToPaint(item, item.Style())))
if (UNLIKELY(!IsVisibleToPaint(item, item.Style())))
return;
NGTextFragmentPainter<NGInlineCursor> text_painter(cursor);
......
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