RFR: 8370498: Improve how Node detects whether a layout property change requires a new layout pass [v2]

Ambarish Rapte arapte at openjdk.org
Mon Oct 27 10:24:46 UTC 2025


On Thu, 23 Oct 2025 16:55:32 GMT, John Hendrikx <jhendrikx at openjdk.org> wrote:

>> This new check is much more accurate to detect whether a parent is currently laying out its children. The previous code almost never worked, resulting in additional unnecessary layouts.
>
> John Hendrikx has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix ToolBarSkinTest
>   
>   Reusing a toolbar as part of several scenes, in combination with the StubToolkit that doesn't handle pulses makes this test fail with the relayout detection fix.

The method `isCurrentLayoutChild()` was introduced with the fix for [JDK-8137252](https://bugs.openjdk.org/browse/JDK-8137252).
With this PR change, the issue [JDK-8137252](https://bugs.openjdk.org/browse/JDK-8137252) recurs.

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

PR Comment: https://git.openjdk.org/jfx/pull/1945#issuecomment-3450525690


More information about the openjfx-dev mailing list