1709735 - [wpt-sync] Sync PR 28853 - [ReplacedNG] Fix relayout boundary logic.

Status: RESOLVED FIXED

(Core :: SVG, task, P4)

Description

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

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

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

b'Ian Kilpatrick <[e-mail]>' wrote:

[ReplacedNG] Fix relayout boundary logic.

We had an issue when a replaced object was a relayout root.

With this change it was possible for a replaced element to clear its
layout results when being laid out as a layout root.

When a RecalcLayoutOverflow was triggered, we'd try and perform the
CloneWithPostLayoutFragments however this would now fail, as the
replaced element fragment "PostLayout()" call would fail, as there was
no updated fragment.

This change makes the replaced element go down the NGBlockNode::Layout
path when laying out from a relayout boundary.

Bug: 1205852, 1205977, 1205971
Change-Id: I7f195817daddd17659564e56f7e0f3b46ec53991

Reviewed-on: https://chromium-review.googlesource.com/2875638
WPT-Export-Revision: 7ea2f02ab3a91820421c0259fe54dee1721f3932

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=8c9458bfd40bc8dd8c73200a632c1227f6d33aa8

Comment 2

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

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

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3327203e7941
[wpt PR 28853] - [ReplacedNG] Fix relayout boundary logic., a=testonly

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/3327203e7941