1597232 - [wpt-sync] Sync PR 20297 - [css-text] There is no shape-result when items overflow under break-word

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 20297 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/20297
Details from upstream follow.

Javier Fernandez <[e-mail]> wrote:

[css-text] There is no shape-result when items overflow under break-word

When the overflow-wrap: 'break-word' feature is used, items overflowing
don't have an associated ShapeResult instance, since we deliberately set
the kNoResultIfOverflow to ignore it because it will be rewound in any
case.

However, in the RewindOverflow function, we incorrectly assumed that
when an item has no associated ShapeResult, it shouldn't be a trailable
text item.

Additionally, this CL removes the clause that implied that items formed
with all trailable spaces should be ignored. Such logic prevented such
kind of items to hang, which is what the spec mandates when pre-wrap is
being used.

Bug: 1003184
Change-Id: Iab930be3ecc8e4dc814be25d902ecedabc70a944

Reviewed-on: https://chromium-review.googlesource.com/1919351
WPT-Export-Revision: bfe2490b94620de010f8ecdee3507911666c5200

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=aa3e7da38ee04d9185912ce403a7bcaed02f9979

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 8 tests

Status Summary

Firefox

PASS: 8

Chrome

PASS: 2
FAIL: 6

Safari

PASS: 8

Links

GitHub PR Head
GitHub PR Base

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/db1d5134a2eb
[wpt PR 20297] - [css-text] There is no shape-result when items overflow under break-word, a=testonly

Comment 4

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/db1d5134a2eb