[wpt-sync] Sync PR 20297 - [css-text] There is no shape-result when items overflow under break-word
Categories
(Core :: Layout: Text and Fonts, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox74 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
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 <jfernandez@igalia.com> 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: Iab930be3ecc8e4dc814be25d902ecedabc70a944Reviewed-on: https://chromium-review.googlesource.com/1919351
WPT-Export-Revision: bfe2490b94620de010f8ecdee3507911666c5200
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=aa3e7da38ee04d9185912ce403a7bcaed02f9979
Assignee | ||
Comment 2•4 years ago
|
||
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
Pushed by wptsync@mozilla.com: 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•4 years ago
|
||
bugherder |
Description
•