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

Kevin Rushforth kcr at openjdk.org
Thu Feb 27 13:51:09 UTC 2025


On Thu, 27 Feb 2025 07:43:59 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:

>> 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.

No, we don't want to add a new public `show` method to `Control` since it is only a very few controls that have the concept of showing a popup.

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

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


More information about the openjfx-dev mailing list