RFR: 8367376: Bad ButtonUI prevents other components from updating when system changes desktop properties [v8]

Alexey Ivanov aivanov at openjdk.org
Wed Oct 8 10:40:19 UTC 2025


On Wed, 8 Oct 2025 05:47:33 GMT, Jeremy Wood <jwood at openjdk.org> wrote:

>> Previously:
>> 
>> If DesktopProperty#updateAllUIs threw an exception, we would never reset the update-pending property to false. This means any subsequent call to `updateUI()` would not attempt to call `updateAllUIs()`
>> 
>> With this change:
>> Subsequent calls to DesktopProperty#updateUI() can still trigger at least one call to updateAllUIs().
>
> Jeremy Wood has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - 8367376: adding @key headful
>    
>    This is in response to the second point here:
>    https://github.com/openjdk/jdk/pull/27205#discussion_r2412548662
>  - 8367376: adding Frame.dispose() in try/finally
>    
>    This is in response to the first point here:
>    https://github.com/openjdk/jdk/pull/27205#discussion_r2412548662

Marked as reviewed by aivanov (Reviewer).

test/jdk/com/sun/java/swing/plaf/DesktopPropertyResetPendingFlagTest.java line 31:

> 29:  * @run main DesktopPropertyResetPendingFlagTest
> 30:  * @key headful
> 31:  */

Suggestion:

/*
 * @test
 * @bug 8367376
 * @key headful
 * @summary DesktopProperty never reset pending status to process new updates
 * @modules java.desktop/sun.swing.plaf
 * @run main DesktopPropertyResetPendingFlagTest
 */

It doesn't really matter, yet usually `@key` comes after the `@bug` tag.

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

PR Review: https://git.openjdk.org/jdk/pull/27205#pullrequestreview-3314186501
PR Review Comment: https://git.openjdk.org/jdk/pull/27205#discussion_r2413408274


More information about the client-libs-dev mailing list