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