RFR: 8367376: Bad ButtonUI prevents other components from updating when system changes desktop properties [v6]
Jeremy Wood
jwood at openjdk.org
Tue Oct 7 17:30:35 UTC 2025
On Tue, 7 Oct 2025 07:22:08 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> Jeremy Wood 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 12 additional commits since the last revision:
>>
>> - 8367376: test all installed L&F's
>>
>> This is in response to:
>> https://git.openjdk.org/jdk/pull/27205#discussion_r2382574446
>> - Merge branch 'master' into JDK-8367376
>> - 8367376: use same try/finally pattern for SynthL&F
>>
>> This is in response to:
>> https://github.com/openjdk/jdk/pull/27205#discussion_r2357719216
>> - 8367376: use same try/finally pattern for MetalL&F
>>
>> This is in response to:
>> https://github.com/openjdk/jdk/pull/27205#discussion_r2357716063
>> - 8367376: remove setLookAndFeel
>>
>> This was probably left over from an earlier draft; this test uses the TestDesktopProperty class so it doesn't rely on/test any particular L&F.
>> - 8367376: rename test file
>>
>> This is in response to:
>> https://github.com/openjdk/jdk/pull/27205#discussion_r2356674752
>> - 8367376: add new line to end of file
>>
>> This is in response to:
>> https://github.com/openjdk/jdk/pull/27205#discussion_r2356666306
>> - 8367376: changing field names
>>
>> This is in response to:
>> https://github.com/openjdk/jdk/pull/27205#discussion_r2356664929
>> - Update test/jdk/com/sun/java/swing/plaf/Test8367376.java
>>
>> Co-authored-by: Alexey Ivanov <alexey.ivanov at oracle.com>
>> - Update test/jdk/com/sun/java/swing/plaf/Test8367376.java
>>
>> Co-authored-by: Alexey Ivanov <alexey.ivanov at oracle.com>
>> - ... and 2 more: https://git.openjdk.org/jdk/compare/db7079dd...1b669f8d
>
> test/jdk/com/sun/java/swing/plaf/DesktopPropertyResetPendingFlagTest.java line 92:
>
>> 90: Toolkit.getDefaultToolkit().getSystemEventQueue().push(newEventQueue);
>> 91:
>> 92: for (UIManager.LookAndFeelInfo info :
>
> UIManager import is missing
Thanks; this is updated.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27205#discussion_r2411359670
More information about the client-libs-dev
mailing list