Closed Bug 1863087 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 42950 - [layout] Simplified layout incorrectly reporting adjoining descendants

Categories

(Core :: Layout: Positioned, task, P4)

task

Tracking

()

RESOLVED FIXED
121 Branch
Tracking Status
firefox121 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Ian Kilpatrick <ikilpatrick@chromium.org> wrote:

[layout] Simplified layout incorrectly reporting adjoining descendants

When a float existing in an inline formatting context, and that
inline formatting context has simplified layout performed, the
simplified layout wouldn't carry across the
"HasAdjoiningObjectDescendants" flag correctly. This had the side
effect of an incorrect cache-hit, resulting in a mis-placed float.

This copies across the field manually.

Bug: 1493929
Change-Id: I09b7e73aac8721f3c9e1f13eec329454e8c18395

Reviewed-on: https://chromium-review.googlesource.com/5005434
WPT-Export-Revision: 0ee6d6a408c6eb3acd94c00cb48f55be8237d508

Component: web-platform-tests → Layout: Positioned
Product: Testing → Core

CI Results

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

Total 1 tests

Status Summary

Firefox

PASS: 1

Chrome

FAIL: 1

Safari

PASS: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/435caabe740a [wpt PR 42950] - [layout] Simplified layout incorrectly reporting adjoining descendants, a=testonly
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/47ab932cfce2 [wpt PR 42950] - [layout] Simplified layout incorrectly reporting adjoining descendants, a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
You need to log in before you can comment on or make changes to this bug.