> On Jan 14, 2020, at 11:50 AM, Ambarish Rapte <ara...@openjdk.java.net> wrote:
>
> On Fri, 10 Jan 2020 00:39:53 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>
>>> I'll review this next week. This seems a fine candidate for openjfx14, so
>>> it (along with a couple other pending reviews that can be for 14) will be a
>>> good test of targeting a PR to the stabilization branch.
>>>
>>> I also request @arapte to review.
>>
>> I should add that it will depend on whether there are any regressions. One
>> thing we do need to be careful of is introducing regressions during rampdown.
>
> The fix looks good to me.
> After this change the steps needed for playing an `Animation` backwards will
> change.
> Earlier this was documented with `Animation.play()` API as below.
> -------------
> To play an `Animation` backwards from the end:
> animation.setRate(negative rate);
> animation.jumpTo(overall duration of animation);
> animation.play();
> -------------
>
> After this PR call to `jumpTo()` won't be needed here.
> So this PR may need a document change for `Animation.play()`.
> Also to note that the behavior of above mentioned three calls remains same
> with the changes in this change.
If the jumpTo isn’t required, then this isn’t this a change in behaviour? I’m
wonder in particular if this has an effect on cycleCount.
If cycleCount is set to 2 does the animation play the same number of times with
or without the jumpTo both before and after this change?
Scott
