RFR: 8349098: TabPane: exception initializing in a background thread [v2]

[Kevin Rushforth]

On Fri, 7 Feb 2025 18:42:01 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> ## Root Cause
>> Animation gets started in a background thread, which causes the animation handler to run in the FX application thread, thus creating simultaneous access to the control's fields (list of children in this case).
>> 
>> ## Solution
>> Skip the animation.
>> 
>> The fix is similar to https://github.com/openjdk/jfx/pull/1698
>
> Andy Goryachev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:
> 
>  - Merge branch 'master' into 8349098.thread.safe.tabpane
>  - skip animation

The fix looks good. I confirm that the newly-enabled test fails without the fix and passes with the fix.

-------------

Marked as reviewed by kcr (Lead).

PR Review: https://git.openjdk.org/jfx/pull/1699#pullrequestreview-2618275147