Skip to main content

Bug Tracker

Side navigation

#10235 closed bug (duplicate)

Opened September 09, 2011 01:30PM UTC

Closed September 20, 2011 04:39AM UTC

Last modified September 20, 2011 04:39AM UTC

Timers deletion race condition causes animations to stop

Reported by: Aitor Guevara <aitorciki@gmail.com> Owned by: gnarf
Priority: low Milestone: None
Component: effects Version: 1.6.3
Keywords: Cc:
Blocked by: Blocking:
Description

jQuery.fx.tick doesn't check if the timers array has been modified inside the timer step function. If a stop() has been registered in an animation callback, the timer will be deleted inside the step function, and tick will delete the wrong timer since the loop counter now points to the next one.

The problem is demonstrated at http://jsfiddle.net/aitorciki/LE9eT/ .

Pull request #495 fixes the problem and adds a test.

Attachments (0)
Change History (5)

Changed September 19, 2011 04:11PM UTC by rwaldron comment:1

owner: → gnarf
status: newassigned

Changed September 19, 2011 04:14PM UTC by rwaldron comment:2

component: unfiledeffects
priority: undecidedlow

Changed September 20, 2011 04:38AM UTC by gnarf comment:3

This might possibly be a dupe of #6641

Changed September 20, 2011 04:39AM UTC by gnarf comment:4

resolution: → duplicate
status: assignedclosed

Confirmed - This behavior is already fixed in master http://jsfiddle.net/LE9eT/4/

Changed September 20, 2011 04:39AM UTC by gnarf comment:5

Duplicate of #6641.