Bug Tracker

Modify

Ticket #10235 (closed bug: duplicate)

Opened 21 months ago

Last modified 20 months ago

Timers deletion race condition causes animations to stop

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

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.

Change History

comment:1 Changed 20 months ago by rwaldron

  • Owner set to gnarf
  • Status changed from new to assigned

comment:2 Changed 20 months ago by rwaldron

  • Priority changed from undecided to low
  • Component changed from unfiled to effects

comment:3 Changed 20 months ago by gnarf

This might possibly be a dupe of #6641

comment:4 Changed 20 months ago by gnarf

  • Status changed from assigned to closed
  • Resolution set to duplicate

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

comment:5 Changed 20 months ago by gnarf

Duplicate of #6641.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.