Commit 8c5e3d06 authored by Kent Tamura's avatar Kent Tamura Committed by Commit Bot

TextControl NG: Do not add scrollbars to text fields

This CL ports LayoutTextControlSingleLine::AllowsNonVisibleOverflow()
to LayoutNGTextControlSingleLine.

This CL has no behavior changes now, but it fixes
baseline-alignment-and-overflow.tentative.html in WPT with
LayoutNGTextField flag.

Bug: 1040826
Change-Id: If21b1b150922229a180293dd9f8c1cceb43ab70a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2501169
Auto-Submit: Kent Tamura <tkent@chromium.org>
Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#821135}
parent 6c4c70b9
...@@ -79,4 +79,10 @@ bool LayoutNGTextControlSingleLine::NodeAtPoint( ...@@ -79,4 +79,10 @@ bool LayoutNGTextControlSingleLine::NodeAtPoint(
return true; return true;
} }
bool LayoutNGTextControlSingleLine::AllowsNonVisibleOverflow() const {
NOT_DESTROYED();
// Do not show scrollbars even if overflow:scroll is specified.
return false;
}
} // namespace blink } // namespace blink
...@@ -37,6 +37,8 @@ class LayoutNGTextControlSingleLine final : public LayoutNGBlockFlow { ...@@ -37,6 +37,8 @@ class LayoutNGTextControlSingleLine final : public LayoutNGBlockFlow {
const HitTestLocation& hit_test_location, const HitTestLocation& hit_test_location,
const PhysicalOffset& accumulated_offset, const PhysicalOffset& accumulated_offset,
HitTestAction hit_test_action) override; HitTestAction hit_test_action) override;
bool AllowsNonVisibleOverflow() const override;
}; };
} // namespace blink } // namespace blink
......
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