RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

Andy Goryachev angorya at openjdk.org
Thu Mar 13 16:21:04 UTC 2025


On Thu, 13 Mar 2025 04:00:42 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:

>> 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 six additional commits since the last revision:
>> 
>>  - Merge remote-tracking branch 'origin/master' into 8350976.menubarskin.thread.safety
>>  - spelling
>>  - use system menu
>>  - cleanup
>>  - possible fix
>>  - test
>
> modules/javafx.controls/src/main/java/javafx/scene/control/skin/MenuBarSkin.java line 233:
> 
>> 231: 
>> 232:         if (Platform.isFxApplicationThread()) {
>> 233:             if (Toolkit.getToolkit().getSystemMenu().isSupported()) {
> 
> You could move this check to the outer scope, because if it evalutes to false, we can skip both branches of `if (Platform.isFxApplicationThread()) {` completely.

not sure what you mean exactly.
Toolkit.getToolkit().getSystemMenu().isSupported() must be called in the fx application thread, and the rest of the changes were done to minimize the structural changes.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1727#discussion_r1993874015


More information about the openjfx-dev mailing list