Bug Tracker

Opened 7 years ago

Closed 7 years ago

#9789 closed bug (wontfix)

If an animation 'complete' callback throws, it will keep getting invoked repeatedly

Reported by: anonymous Owned by:
Priority: low Milestone: 1.next
Component: effects Version: 1.6.2
Keywords: Cc:
Blocked by: Blocking:

Description

See https://bugzilla.mozilla.org/show_bug.cgi?id=669107#c4

Basically, if a 'complete' callback throws, its corresponding animation will keep going, and any later animations won't be processed correctly at all.

Change History (1)

comment:1 Changed 7 years ago by timmywil

Component: unfiledeffects
Priority: undecidedlow
Resolution: wontfix
Status: newclosed

This cannot be changed unless we want to go back to the old queue order. However, if callbacks do not throw errors (which they shouldn't after debugging), everything works fine so I'm not sure having a disordered queue is worth the fix. Ideally, we could use a try {} finally {}, but this causes problems in IE if there's no catch (which we can't add for debugging reasons). And setTimeout(f, 0) would be unpredictable. If we want to maintain consistent behavior across browsers, it should probably stay as is.

Note: See TracTickets for help on using tickets.