On Fri, 14 Feb 2025 16:23:58 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>> Although, having said that, the purpose of the cleanup is to clean up after >> the animation. So a better fix might be to put the entire if-else inside an >> `if (Platform.isFxApplicationThread())` test. > > Actually, no. It also calls requestLayout. I think the fix is correct as is. Right. The change prevents from _starting_ the animation if these code paths are entered in a background thread. With the change, the code follows an animation-off path. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1699#discussion_r1956518465
