Open Bug 1353212 Opened 7 years ago Updated 1 year ago

Flush only transform animation styles when we handle events

Categories

(Core :: DOM: Animation, enhancement, P3)

Product:
Core
Component:
DOM: Animation
Type:
enhancement

Tracking

()

People

(Reporter: hiro, Unassigned)

Details

When we handle events, we flush all throttled animations [1] to get correct up-to-date position for the events (IIUC).

But, IIUC, we just need to update throttled transform animations for the case, we don't need to update opacity animations and other throttled animations (e.g. position is unchanged due to step functions).

[1] https://hg.mozilla.org/mozilla-central/file/b5d8b27a7537/layout/base/PresShell.cpp#l7376
I understand this bug is not a right place but there is no right bug yet.

Currently we do check HasPendingStyleUpdates() [1] in ServoRestyleManager::UpdateOnlyAnimationStyles(), but it should be HasThrottledStyleUpdates() for now.

[1] https://hg.mozilla.org/mozilla-central/file/a418121d4625/layout/base/ServoRestyleManager.cpp#l872
Severity: normal → S3

Is this bug fixed or not? I'm also facing such issue from couple of days.
https://zarahairstyles.com/pink-hair-color-ideas/

You need to log in before you can comment on or make changes to this bug.