Open Bug 1865051 Opened 2 years ago Updated 1 year ago

Consider to call js::NotifyAnimationActivity from nsRefreshDriver::Tick

Categories

(Core :: JavaScript: GC, enhancement, P2)

Product:
Core
Component:
JavaScript: GC
Type:
enhancement

Tracking

()

People

(Reporter: smaug, Unassigned, NeedInfo)

References

(Blocks 2 open bugs)

Details

Right now the method is called from Window::RequestAnimationFrame https://searchfox.org/mozilla-central/rev/f78093864e287014db7ac9383bb76c45edbf8559/dom/base/nsGlobalWindowInner.cpp#3543 and few other places.
It might be better to move the calls to nsRefreshDriver::Tick, or have additional call in Tick, to catch other sorts of "animations".
Perhaps one could iterate through style and layout observers and notify the relevant realms https://searchfox.org/mozilla-central/rev/f78093864e287014db7ac9383bb76c45edbf8559/layout/base/nsRefreshDriver.h#701-702

Blocks: sm-runtime
Component: JavaScript Engine → JavaScript: GC
Flags: needinfo?(sphink)

Not sure this bug belongs to the JavaScript engine … nor that we have a call in what lands in the Layout component.

Blocks: sm-meta
Severity: -- → N/A
Type: task → enhancement
Priority: -- → P2
No longer blocks: sm-meta
You need to log in before you can comment on or make changes to this bug.