The Mozilla standards position for that is positive. According to the MDN Browser Compat Data, all other browsers are shipping requestVideoFrameCallback and cancelVideoFrameCallback already.

See https://wicg.github.io/video-rvfc/ for standard details.

Backed out for causing mochitest failures related to animations on compositor.

• Backout link
• Push with failures
• Failure Log:

[task 2024-07-24T20:15:54.287Z] 20:15:54     INFO - TEST-START | dom/animation/test/chrome/test_animation_performance_warning.html
[task 2024-07-24T20:15:54.288Z] 20:15:54     INFO - GECKO(1306) | [Parent 1306, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005 (NS_ERROR_FAILURE): file /builds/worker/checkouts/gecko/chrome/nsChromeRegistry.cpp:182
[task 2024-07-24T20:15:54.288Z] 20:15:54     INFO - GECKO(1306) | [Parent 1306, Main Thread] WARNING: 'NS_FAILED(rv)', file /builds/worker/checkouts/gecko/chrome/nsChromeProtocolHandler.cpp:73
[task 2024-07-24T20:15:54.289Z] 20:15:54     INFO - GECKO(1306) | [Parent 1306, Main Thread] WARNING: Failed to retarget HTML data delivery to the parser thread.: file /builds/worker/checkouts/gecko/parser/html/nsHtml5StreamParser.cpp:1215
[task 2024-07-24T20:15:54.290Z] 20:15:54     INFO - TEST-INFO | started process screentopng
[task 2024-07-24T20:15:54.516Z] 20:15:54     INFO - TEST-INFO | screentopng: exit 0
[task 2024-07-24T20:15:54.516Z] 20:15:54     INFO - TEST-UNEXPECTED-FAIL | dom/animation/test/chrome/test_animation_performance_warning.html | animations on compositor - animations on compositor: assert_equals: runningOnCompositor property should match expected true but got false
[task 2024-07-24T20:15:54.517Z] 20:15:54     INFO - TEST-PASS | dom/animation/test/chrome/test_animation_performance_warning.html | Bug 1196114 - Test metadata related to which animation properties are running on the compositor - Bug 1196114 - Test metadata related to which animation properties are running on the compositor: Elided 67 passes or known failures.

Deal with throttled docs only once upfront, since
ShouldThrottleFrameRequests is kind of expensive (doing a full
IntersectionObserver check).

Add proper spec quotes too, while at it. This also fixes the failure
that got the previous patch backed out.

This fixes rusttests. Otherwise we include HTMLVideoElement.h from
Document.h, which includes a bunch of media headers, which causes
rusttest failures because
https://searchfox.org/mozilla-central/rev/d353cfa1fbd207e13dc974f30e5f88535a4303ae/dom/media/platforms/EncoderConfig.h#95
hits https://github.com/rust-lang/rust-bindgen/issues/380.

We could hide those types from rust but it seems slightly nicer to keep
Document.h lean, since it's included in a gazillion places.

Backed out for causing webgpu failures.

Backout link: https://hg.mozilla.org/integration/autoland/rev/c8b2755739da05a6986c6d43cdd28718b4ac390d

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=8acda0361e97220b903120da806455c114180678&selectedTaskRun=FWLxSSnvR0iuB1AXQ51wSA.0

Failure log: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=8acda0361e97220b903120da806455c114180678&selectedTaskRun=FWLxSSnvR0iuB1AXQ51wSA.0

(In reply to Atila Butkovits from comment #10)

Backed out for causing webgpu failures.

Backout link: https://hg.mozilla.org/integration/autoland/rev/c8b2755739da05a6986c6d43cdd28718b4ac390d

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=8acda0361e97220b903120da806455c114180678&selectedTaskRun=FWLxSSnvR0iuB1AXQ51wSA.0

Failure log: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=8acda0361e97220b903120da806455c114180678&selectedTaskRun=FWLxSSnvR0iuB1AXQ51wSA.0

Perfherder has detected a browsertime performance change from push b81788b1a72473564135c6dd38ffa0e89a21d308.

Regressions:

Improvements:

As author of one of the patches included in that push, we need your help to address this regression.
Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests. Please follow our guide to handling regression bugs and let us know your plans within 3 business days, or the patch(es) may be backed out in accordance with our regression policy.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a sheriff to do that for you.

You can run these tests on try with ./mach try perf --alert 1489

For more information on performance sheriffing please see our FAQ.

Backed out for causing webcodecs failures

Backout link: https://hg.mozilla.org/integration/autoland/rev/de8daa863ecfe1b5b6f081143389f87ea0189a6a

Push with failures

Failure log -> TEST-UNEXPECTED-TIMEOUT | /webcodecs/videoFrame-canvasImageSource.html

https://hg.mozilla.org/mozilla-central/rev/40274941ace1
https://hg.mozilla.org/mozilla-central/rev/6418025eb4ec
https://hg.mozilla.org/mozilla-central/rev/65d739fc547d

Woot! \o/