ngAnimate + $$RAFProvider : crashes when browser doesn't support requestAnimationFrame or is prefixed with 'moz'

[atomrc]

The RAFProvider is adding a supported property to indicate whether the browser supports or not the requestAnimationFrame (see

angular.js/src/ng/raf.js

Line 18 in 04d7317

raf.supported = !!requestAnimationFrame;

) but the ngAnimate doesn't check this property before using it (see https://github.com/angular/angular.js/blob/master/src/ngAnimate/animate.js#L261).

I would be happy to make a pull request on this but my question is :
is it best to add a fallback on a timeout directly in the RAFProvider or to check for the supportedproperty every time it is used ?

I think just adding the mozRequestAnimationFrame would be a good start.

[matsko]

Yes let's add mozRequestAnimationFrame. All browsers that support CSS animations/transitions also support RAF, therefore, there is no reason to fallback to timeout anymore.

I didn't think that Firefox still needed it. I guess the error showed up on an older version of FF.

[matsko]

Can you verify that adding mozRequestAnimationFrame and mozCancelAnimationFrame fixes the issue?

[atomrc]

Yes indeed the error occurs in an old version of FF (20).
The fact is that if the browser doesn't support RAF (I am thinking of IE8 for example) the code will simply crash.
What is weird is that there is the property supported (false if the browser doesn't support rAF) but it is never checked wherever the rAF service is used.