New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 648114 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocked on:
issue 723099



Sign in to add a comment

Animations don't play after calling pause() then play() in separate script tasks

Project Member Reported by alancutter@chromium.org, Sep 19 2016

Issue description

Test case: https://rawgit.com/web-animations/web-animations-demos/4f26723f1cf39dd10a70698ed00a805f658ffdee/phone/index.html
Repro steps:
 - Click the bottom bar, wait for slide up animation to end.
 - Click the bar again.

Expected:
Bar should slide back down.

Actualy:
Bar only slides down when clicked again or after toggling tab focus.

The tab focus suggests this is related to compositor/main thread animation co-ordination.
 
Cc: thorogood@chromium.org
Status: Assigned (was: Untriaged)
Created more minimal test case: https://jsfiddle.net/2b8ykcu0/

This seems to require a combination of:
 - Composited animations
 - Pausing then playing in separate script tasks
 - Using will-change

Needs further investigation.
Minified test case even further: https://jsfiddle.net/7rt8x76g/
After speaking with loyso it seems that calls to AnimationDelegate::notifyAnimationStarted() in cc animations land is likely where the bug lies.
Labels: Hotlist-Interop
It sounds like this is an interop bug - ie Firefox's Web Animations doesn't do this (sorry, my Firefox install is corrupt so I can't test it). 

If not, please remove the Hotlist-Interop label.
Blockedon: 723099
Cc: alancutter@chromium.org
Owner: jiameng@chromium.org
The problem came from animation compositing (see  issue 723099  for details). The compositing team is putting in a patch to fix it, and I confirm it also solves the problem reported in this ticket: I patched in their cl and ran the test animations mentioned in this ticket. I didn't see any problem any more.

The compositing cl is still in codereview. I'll update as soon as it's landed.
Status: Fixed (was: Assigned)
The compositing cl has been submitted, which also fixed the animations bug reported in this ticket.

Sign in to add a comment