Commit 7f37acf7 authored by Yoshifumi Inoue's avatar Yoshifumi Inoue Committed by Commit Bot

Get rid of unused variable LayoutBlockFlow::AddVisualOverflowFromInlineChildren()

This patch gets rid of unused variable |end_padding| from |LayoutBlockFlow::
AddVisualOverflowFromInlineChildren()| for improving code health.

Change-Id: Ib2c578a9815c22a197e8b9eb409839fefbc53735
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2228025
Auto-Submit: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: default avatarKent Tamura <tkent@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#774981}
parent 36f8ca1e
...@@ -2403,13 +2403,6 @@ bool LayoutBlockFlow::GeneratesLineBoxesForInlineChild(LayoutObject* inline_obj) ...@@ -2403,13 +2403,6 @@ bool LayoutBlockFlow::GeneratesLineBoxesForInlineChild(LayoutObject* inline_obj)
} }
void LayoutBlockFlow::AddVisualOverflowFromInlineChildren() { void LayoutBlockFlow::AddVisualOverflowFromInlineChildren() {
LayoutUnit end_padding = HasOverflowClip() ? PaddingEnd() : LayoutUnit();
// FIXME: Need to find another way to do this, since scrollbars could show
// when we don't want them to.
if (HasOverflowClip() && !end_padding && GetNode() &&
IsRootEditableElement(*GetNode()) && StyleRef().IsLeftToRightDirection())
end_padding = LayoutUnit(1);
if (const NGPaintFragment* paint_fragment = PaintFragment()) { if (const NGPaintFragment* paint_fragment = PaintFragment()) {
for (const NGPaintFragment* child : paint_fragment->Children()) { for (const NGPaintFragment* child : paint_fragment->Children()) {
if (child->HasSelfPaintingLayer()) if (child->HasSelfPaintingLayer())
......
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