[wpt-sync] Sync PR 28488 - Consider HasOverflow true as valid scenario for HandleAtomicInline
Categories
(Core :: Layout: Text and Fonts, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox90 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 28488 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/28488
Details from upstream follow.
b'Javier Fern\xc3\xa1ndez Garc\xc3\xada-Boente <jfernandez@igalia.com>' wrote:
Consider HasOverflow true as valid scenario for HandleAtomicInline
The table caption element always have 0 available space, since its text
consumes all of it. Hence, adding an empty span with negative margins
may imply the line to overflow. If we add some atomic-inline elements
in such caption, we might need to process them under the scenario of a
NGInlineInfo which has true sa HasOverflow flag.In the current code, there is an assertion that under kContinue state
and negative remaining_width (implying overflow) it's not valid to have
the line-info's HasOverflow flag to true.I believe this assertion is contradictory and that this is indeed a
valid scenario, hence, this CL removes such assertion.Bug: 1197974
Change-Id: I80c48e5dffe5e5ee25fcb4cc15db02d584adc4b9
Reviewed-on: https://chromium-review.googlesource.com/2821314
WPT-Export-Revision: ecea8703f2bca87ec790e122ee428d54323266da
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 1•3 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=c409ba5503f029e1041ed60d3daa6d2a319b1a69
Assignee | ||
Comment 2•3 years ago
|
||
CI Results
Ran 14 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 1 tests
Status Summary
Firefox
PASS: 1
Chrome
PASS: 1
Safari
PASS: 1
Links
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8071e722c442 [wpt PR 28488] - Consider HasOverflow true as valid scenario for HandleAtomicInline, a=testonly
Comment 4•3 years ago
|
||
bugherder |
Description
•