New issue
Advanced search Search tips

Issue 805323 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Composited implicit keyframe animations jump back to start position

Project Member Reported by flackr@chromium.org, Jan 24 2018

Issue description

Chrome Version: 63.0.3239.140 (Official Build) (64-bit)
OS: Chrome

What steps will reproduce the problem?
(1) Enable experimental web platform features at chrome://flags/#enable-experimental-web-platform-features
(2) Navigate to http://jsbin.com/fifopew/edit?html,css,js,output
(3) Click (1) more than once, note that the composited animation starts from the beginning every time even though its main thread value stays to the right. See notes on the page for other strange issues.

What is the expected result?
The green box should never jump.

What happens instead?
Clicking the various positions (1, 2, 3) either again or one after the other can cause jumps in the green box's position.

Please use labels and text to provide additional information.


For graphics-related bugs, please copy/paste the contents of the about:gpu
page at the end of this report.

 
Owner: alancutter@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 25 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/df884f79de135d1fae253c28da6b96413294106e

commit df884f79de135d1fae253c28da6b96413294106e
Author: Alan Cutter <alancutter@chromium.org>
Date: Thu Jan 25 05:12:22 2018

Restrict neutral keyframes from being composited on top of finished animations

This CL prevents animations from running on the compositor when they contain
neutral keyframes and apply on top of existing animaions on the same property.

Bug:  805323 
Change-Id: Iba19882454211f127358b78fcbfa32bfdf183048
Reviewed-on: https://chromium-review.googlesource.com/884781
Reviewed-by: Robert Flack <flackr@chromium.org>
Commit-Queue: Alan Cutter <alancutter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531814}
[add] https://crrev.com/df884f79de135d1fae253c28da6b96413294106e/third_party/WebKit/LayoutTests/animations/stacked-neutral-keyframe-expected.html
[add] https://crrev.com/df884f79de135d1fae253c28da6b96413294106e/third_party/WebKit/LayoutTests/animations/stacked-neutral-keyframe.html
[modify] https://crrev.com/df884f79de135d1fae253c28da6b96413294106e/third_party/WebKit/Source/core/animation/CompositorAnimations.cpp
[modify] https://crrev.com/df884f79de135d1fae253c28da6b96413294106e/third_party/WebKit/Source/core/animation/EffectModel.h
[modify] https://crrev.com/df884f79de135d1fae253c28da6b96413294106e/third_party/WebKit/Source/core/animation/KeyframeEffectModel.cpp
[modify] https://crrev.com/df884f79de135d1fae253c28da6b96413294106e/third_party/WebKit/Source/core/animation/KeyframeEffectModel.h

Status: Fixed (was: Assigned)

Sign in to add a comment