Closed Bug 2021539 Opened 2 months ago Closed 2 months ago

[wpt-sync] Sync PR 58304 - Treat static video controls as replaced normal-flow stacking

Categories

(Core :: DOM: Core & HTML, task, P4)

Product:
Core
Component:
DOM: Core & HTML
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
150 Branch
Tracking Flags:
Tracking Status
firefox150 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

jj <jj@imput.net> wrote:

Treat static video controls as replaced normal-flow stacking

Static video controls create stacked paint layers, which paint after
inline siblings, causing positioned elements (e.g. buttons) placed on
top of the video to be painted over by the controls.

Fix this by treating static \<video controls> as a replaced normal-flow
stacking element.

The hit-test and paint child-skip in PaintLayer is narrowed from
IsReplacedNormalFlowStacking() to IsSVGForeignObject(), since the skip
logic relies on SVG-specific coordinate transforms that don't apply to
video.

Fixed: 40591804
Change-Id: Ib0489f88fc1452ceb8a34e3925809eb0ca73b70e
Reviewed-on: https://chromium-review.googlesource.com/7617187
WPT-Export-Revision: 39b5015d09ace416d84ff9116dca7997da0436e5

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → INVALID
Status: RESOLVED → REOPENED
Resolution: INVALID → ---

CI Results

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

Total 10 tests

Status Summary

Firefox

OK : 1[GitHub] 3[Gecko-android-em-14-x86_64-debug-geckoview, Gecko-android-em-14-x86_64-lite-opt-geckoview, Gecko-android-em-14-x86_64-opt-geckoview, Gecko-linux2404-64-debug, Gecko-linux2404-64-opt, Gecko-windows11-32-24h2-debug, Gecko-windows11-32-24h2-opt, Gecko-windows11-64-24h2-debug, Gecko-windows11-64-24h2-opt]
PASS: 2[GitHub] 9[Gecko-windows11-32-24h2-debug, Gecko-windows11-32-24h2-opt] 15[Gecko-android-em-14-x86_64-debug-geckoview, Gecko-android-em-14-x86_64-lite-opt-geckoview, Gecko-android-em-14-x86_64-opt-geckoview, Gecko-linux2404-64-debug, Gecko-linux2404-64-opt, Gecko-windows11-64-24h2-debug, Gecko-windows11-64-24h2-opt]
FAIL: 6

Chrome

OK : 1
FAIL: 2

Safari

OK : 1
PASS: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

Tests Disabled in Gecko Infrastructure

Pushed by wptsync@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/a8911b543e8e https://hg.mozilla.org/integration/autoland/rev/878cc1504343 [wpt PR 58304] - Treat static video controls as replaced normal-flow stacking, a=testonly

https://hg.mozilla.org/mozilla-central/rev/878cc1504343

Status: REOPENED → RESOLVED
Closed: 2 months ago → 2 months ago
status-firefox150: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 150 Branch
You need to log in before you can comment on or make changes to this bug.