Closed Bug 1888723 Opened 1 year ago Closed 1 year ago

[wpt-sync] Sync PR 45435 - [anchor] Sort candidate list before handing off to OOFLayoutPart.

Categories

(Core :: CSS Parsing and Computation, task, P4)

task

Tracking

()

RESOLVED FIXED
126 Branch
Tracking Status
firefox126 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Ian Kilpatrick <ikilpatrick@chromium.org> wrote:

[anchor] Sort candidate list before handing off to OOFLayoutPart.

The order of the candidates now is important, when it wasn't
previously. The order is only important when there is a child that
has an anchor within it, so to mitigate the performance, only sort
when that is true.

The order can get inconsistent for a variety of reasons, the testcase
is when we insert them in the incorrect order during simplified layout
(as we don't keep the OOFs in the same list as the child fragments).

Fixed: 330756838
Change-Id: I7e4e08d104c5a75e96b6250ba234183ca3ab458f

Reviewed-on: https://chromium-review.googlesource.com/5404921
WPT-Export-Revision: 10490af8886929602868f0b1540801a9188f6220

Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core

CI Results

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

Total 1 tests

Status Summary

Firefox

FAIL: 1

Chrome

FAIL: 1

Safari

FAIL: 1

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1a5365a233a3 [wpt PR 45435] - [anchor] Sort candidate list before handing off to OOFLayoutPart., a=testonly https://hg.mozilla.org/integration/autoland/rev/a10334295dd5 [wpt PR 45435] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 126 Branch
You need to log in before you can comment on or make changes to this bug.