1799064 - Avoid redundant PostUpdate in SetTimelineNoUpdate()

Status: RESOLVED FIXED

(Core :: CSS Transitions and Animations, enhancement, P3)

Description

[Boris Chiou [:boris]]

We call SetCurrentTime procedure in SetTimelineNoUpdate, per the spec. However, this function shouldn't PostUpdate(), so we should factor out the part we need for SetTimelineNoUpdate().

Comment 1

[Boris Chiou [:boris]]

Attachment: Bug 1799064 - Avoid potential call PostUpdate() twice in SetTimeline().

Basically, in SetTimelineNoUpdate(), we shouldn't call PostUpdate(). So we
shouldn't call SetCurrentTime() directly.

There is no behavior change. Just make sure we don't PostUpdate twice
while updating an existing CSS animation via changing animation-timeline.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bfdefd57b52e
Avoid potential call PostUpdate() twice in SetTimeline(). r=emilio

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/bfdefd57b52e