[wpt-sync] Sync PR 26455 - Fix |LayoutNGBlockFlowMixin::InlineBlockBaseline|
Categories
(Core :: CSS Parsing and Computation, task, P4)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox84 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 26455 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/26455
Details from upstream follow.
b'Koji Ishii <kojii@chromium.org>' wrote:
Fix |LayoutNGBlockFlowMixin::InlineBlockBaseline|
r824961 crrev.com/c/2515933 copied the code from |LayoutBlock|
but the code in |LayoutBlockFlow| that prevents it from
running when |ChildrenInline| was not copied. This patch adds
the check.Note, the test creates |LayoutNGCustom| in legacy, and adds a
|LayoutTextFragment| as a child of the |LayoutNGCustom|. Not
to blockify the child is likely an issue in |LayoutNGCustom|,
but this case could probably be reproduced by inline custom
layout or other inline objects that forces NG.Bug: 1146789
Change-Id: I7ffacc2465ad205d13702dfda03433cc50c423f9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526702
Commit-Queue: Koji Ishii \<kojii@chromium.org>
Reviewed-by: Morten Stenshorne \<mstensho@chromium.org>
Reviewed-by: Yoshifumi Inoue \<yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#825714}
| Assignee | ||
Updated•5 years ago
|
| Assignee | ||
Comment 1•5 years ago
|
||
| Assignee | ||
Comment 2•5 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
Comment 4•5 years ago
|
||
| bugherder | ||
Description
•