RFR: 8350048: Enforce threading restrictions for show and hide methods in Window, Control, and Skin [v2]

Ambarish Rapte arapte at openjdk.org
Thu Feb 27 07:47:05 UTC 2025


On Wed, 26 Feb 2025 23:39:27 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> - enforced fx application thread
>> - added a headful test `TestThreadingRestrictions`
>> 
>> ## Note to the Reviewers
>> 
>> To avoid merge conflicts, the preferred order of integrations:
>> 
>> #1697 
>> #1713 
>> #1717
>
> Andy Goryachev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains four commits:
> 
>  - Merge remote-tracking branch 'origin/master' into 8350048.enforce
>  - fixed node init test
>  - all tests
>  - initial test

modules/javafx.controls/src/main/java/javafx/scene/control/ChoiceBox.java line 491:

> 489:             setShowing(true);
> 490:         }
> 491:     }

Would it be a good idea to move the check `Toolkit.getToolkit().checkFxUserThread();` to a new method `show()` in Parent class `Control`? And may be similarly to Parent classes of other classes.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1717#discussion_r1973034032


More information about the openjfx-dev mailing list