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

[webkit-line-clamp] Ensure that a -w-l-c box creates a new FC.

As above - this fixes:
external/wpt/css/css-overflow/webkit-line-clamp-004.html

We weren't performing the correct relayout if the lines exactly matched
the requested number of lines.

Bug: 993813
Change-Id: I4ab3a702161bcb8e8cd10fc8ea6837931ebac16d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2224601
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#774000}
parent dda0a01b
...@@ -4330,6 +4330,7 @@ bool LayoutBlockFlow::CreatesNewFormattingContext() const { ...@@ -4330,6 +4330,7 @@ bool LayoutBlockFlow::CreatesNewFormattingContext() const {
IsDocumentElement() || IsGridItem() || IsWritingModeRoot() || IsDocumentElement() || IsGridItem() || IsWritingModeRoot() ||
IsMathItem() || StyleRef().Display() == EDisplay::kFlowRoot || IsMathItem() || StyleRef().Display() == EDisplay::kFlowRoot ||
ShouldApplyPaintContainment() || ShouldApplyLayoutContainment() || ShouldApplyPaintContainment() || ShouldApplyLayoutContainment() ||
StyleRef().IsDeprecatedWebkitBoxWithVerticalLineClamp() ||
StyleRef().SpecifiesColumns() || StyleRef().SpecifiesColumns() ||
StyleRef().GetColumnSpan() == EColumnSpan::kAll) { StyleRef().GetColumnSpan() == EColumnSpan::kAll) {
// The specs require this object to establish a new formatting context. // The specs require this object to establish a new formatting context.
......
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