RFR: 8371859: Dialog unnecessarily invokes DialogPane.requestLayout()

[Andy Goryachev]

On Thu, 13 Nov 2025 21:24:25 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:

> The implementation of `Dialog.dialogPane` adds a `ListChangeListener` to `DialogPane.getButtonTypes()`, and calls `DialogPane.requestLayout()` from the listener.
> 
> This is not needed, because the `DialogPane.createButtonBar()` method already specifies that implementers are required to observe changes of `DialogPane.getButtonTypes()`, and relayout the button bar as needed.
> 
> I've added a test to verify that changing the button types indeed requests a layout on the `DialogPane`.

Despite the questions about the test, the fix seems right - I see no regression.

please wait the customary 24 hrs before integrating, in case someone else wants to take a look at it.

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

Marked as reviewed by angorya (Reviewer).

PR Review: https://git.openjdk.org/jfx/pull/1973#pullrequestreview-3462114044
PR Comment: https://git.openjdk.org/jfx/pull/1973#issuecomment-3530148442