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

Issue 620137 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug

Blocking:
issue 591607



Sign in to add a comment

web animations with "easing" freeze tab with large numbers

Project Member Reported by michae...@chromium.org, Jun 14 2016

Issue description

Version: 51.0.2704.84
OS: Linux

If an animation's first keyframe has an 'easing' other than 'linear', setting the animation's playbackRate and duration to something huge pegs the CPU and locks up the tab.

Example (may freeze): https://jsfiddle.net/20cn4z9t/1/

Is this WAI?


var d = document.querySelector('div');
var a = d.animate([{
  height: '100px',
  easing: 'ease-out',
}, {
  height: '200px',
}], {
  duration: 1e+308, // assumes 1e+308 < Number.MAX_VALUE < 1e+309});
});

a.playbackRate = 1e+20;

setTimeout(function(){ 
  d.innerText = 'One second later'; // doesn't get called
}, 1000);

 
Labels: OS-All
Status: Available (was: Untriaged)
AnimationUtilities.h:
inline double accuracyForDuration(double duration)
{
    return 1.0 / (200.0 * duration);
}

This is probably returning 0 and causing infinite loops.

Comment 2 by suzyh@chromium.org, Jun 15 2016

Labels: Update-quarterly

Comment 3 by loyso@chromium.org, Jun 15 2016

Blocking: 591607

Comment 4 by suzyh@chromium.org, Jun 15 2016

Labels: -Update-quarterly Update-Monthly

Comment 5 by loyso@chromium.org, Jun 15 2016

Probably we want to erase that adaptive accuracy in blink and make it fixed as in cc. 

Comment 6 by suzyh@chromium.org, Jul 4 2016

Labels: Hotlist-Squash-A-Bug

Comment 7 by loyso@chromium.org, Aug 9 2016

Cc: loyso@chromium.org

Comment 8 by loyso@chromium.org, Aug 24 2016

Cc: -loyso@chromium.org
Owner: loyso@chromium.org
Status: Assigned (was: Available)

Comment 9 by loyso@chromium.org, Aug 25 2016

Status: Started (was: Assigned)

Comment 11 by loyso@chromium.org, Aug 25 2016

Status: Fixed (was: Started)

Sign in to add a comment