Comment on attachment 125673[details]
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=125673&action=review
Apart from the naming nitpick, this is an r+ for me on the TextureMapper part.
smfr/others, are you ok with the common part?
> Source/WebCore/platform/graphics/texmap/TextureMapperAnimation.cpp:30
> +static bool animationValueIsReverse(Animation::AnimationDirection direction, int loopCount)
<nitpicks/>
name -> shouldReverseAnimationValue

Comment on attachment 125880[details]
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=125880&action=review
I think we need a test for "reverse" as well. I could only see one for "alternate-reverse". r- for the missing test, but otherwise looks good.
NOTE: Apple will need to raise a followup bug because our hardware animation won't support this directly.
> LayoutTests/animations/animation-direction-alternate-reverse.html:53
> +<!-- Test animation-direction: alternate -->
Nit: it's now alternate-reverse