1916510 - [wpt-sync] Sync PR 47944 - Reland "Support z-index on page margin boxes."

Status: RESOLVED FIXED

(Core :: Layout, task, P4)

Description

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

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

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

Morten Stenshorne <[e-mail]> wrote:

Reland "Support z-index on page margin boxes."

This time without causing
https://ci.chromium.org/ui/p/chromium/builders/ci/lacros64-archive-rel/119842/overview

Original change's description:

Revert "Support z-index on page margin boxes."

This reverts commit ec8a3455717b6e1bec96c86d44fbce326aab79e3.

Reason for revert: broke the tree

Original change's description:

Support z-index on page margin boxes.

If page margin boxes overlap with each other (which should be rare and
normally not ideal and probably ugly), z-index may be used to specify
paint order (default paint order being clockwise from top left corner
margin box).

This CL implements that, except that we cannot control the actual
paginated content (the page area), which will be painted on top of
everything else unconditionally, since it's in a separate layer. See
crbug.com/363031541

Move paginated root / page container paint code into a separate function
(rather than adding more code to the main paint machinery), and add
support for z-index on children there.

Also make sure that page border box fragments are set up to paint
atomically, just like page margin box fragmets, or the page border would
be painted in the background phase, and margin borders in the foreground
phase (which is the phase that paints everything when painting is
atomic).

Bug: 40341678
Change-Id: I480ebf8e8fc2478dfa28163cd001b22371295270
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5828544
Commit-Queue: Morten Stenshorne \<[e-mail]>
Reviewed-by: Xianzhu Wang \<[e-mail]>
Reviewed-by: Ian Kilpatrick \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1349371}

Bug: 40341678
Change-Id: I547f5709a29eef90b865edfea159bb440e762593
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5828933
Bot-Commit: Rubber Stamper \<[e-mail]>
Commit-Queue: Nina Satragno \<[e-mail]>
Owners-Override: Nina Satragno \<[e-mail]>
Reviewed-by: Frank Liberato \<[e-mail]>
Owners-Override: Frank Liberato \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1349377}

Bug: 40341678
Change-Id: Id7c382af4e6e4668b5b96cfbb5cb7151c6ef85f6
Reviewed-on: https://chromium-review.googlesource.com/5829205
WPT-Export-Revision: b396463ea32f00b592138621d1129a862a19c4b1

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=791d0143df580ab0b264608509a7df6edff9c39a

Comment 2

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

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=9aa7da977558b22cbe1ab4b890c90c2762a69880

Comment 3

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

CI Results

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

Total 1 tests

Status Summary

Firefox

FAIL: 1

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

• /css/css-page/margin-boxes/paint-order-003-print.html [wpt.fyi]: FAIL

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/13cbcc728c6d
[wpt PR 47944] - Reland "Support z-index on page margin boxes.", a=testonly
https://hg.mozilla.org/integration/autoland/rev/f29b81ca9ac5
[wpt PR 47944] - Update wpt metadata, a=testonly

Comment 5

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/13cbcc728c6d
https://hg.mozilla.org/mozilla-central/rev/f29b81ca9ac5