Current rendering in NG

Deleted: clusters-insufficient-text-expected.png 42.6 KB

	clusters-insufficient-text-expected.png 42.6 KB View Download

Rendering with the legacy engine

Deleted: clusters-insufficient-text-expected (1).png 86.2 KB

	clusters-insufficient-text-expected (1).png 86.2 KB View Download

The current rendering in NG doesn't do what the test intended for, but the test and the ref match, so it still passes. However, this will no longer be the case once we add support for text-autosizing in NG.

This CL will break the test in question (for NG): https://chromium-review.googlesource.com/c/chromium/src/+/1083191

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/95047205a9c7e6ad59b0d6478f7e4142f307802b

commit 95047205a9c7e6ad59b0d6478f7e4142f307802b
Author: Morten Stenshorne <mstensho@chromium.org>
Date: Tue Jun 05 20:14:25 2018

Don't depend on buggy behavior for fast/text-autosizing/clusters-insufficient-text.html

The legacy layout engine has a bug that's not present in LayoutNG,
making it impossible for LayoutNG to pass
fast/text-autosizing/clusters-insufficient-text.html .

The bug: If a new formatting context in-flow block-level object gets
pushed down by a float, strange things happen in legacy layout if this
object is a writing mode root. Rather than using the initial containing
block as guidance for its inline size, it behaves as if inline-size were
set to min-content. So just specify height (inline-size) as min-content
in the test, to get identical behavior in the legacy engine and
LayoutNG.

Bug:  848874 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I5928318f17ad3105350bd38d3b81cbe5c35819d0
Reviewed-on: https://chromium-review.googlesource.com/1086996
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#564636}
[modify] https://crrev.com/95047205a9c7e6ad59b0d6478f7e4142f307802b/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[modify] https://crrev.com/95047205a9c7e6ad59b0d6478f7e4142f307802b/third_party/WebKit/LayoutTests/fast/text-autosizing/clusters-insufficient-text-expected.html
[modify] https://crrev.com/95047205a9c7e6ad59b0d6478f7e4142f307802b/third_party/WebKit/LayoutTests/fast/text-autosizing/clusters-insufficient-text.html