RFR: 8341286: TreeView: visual artifacts when setting new root with null value

[Ziad El Midaoui]

On Mon, 28 Jul 2025 22:20:52 GMT, Ziad El Midaoui <zelmidaoui at openjdk.org> wrote:

>> modules/javafx.controls/src/main/java/javafx/scene/control/skin/TreeCellSkin.java line 363:
>> 
>>> 361:     private void updateDisclosureNode() {
>>> 362: //        JDK-8341281 : Root TreeItem with null value breaks TreeTableView
>>> 363: //        if (getSkinnable().isEmpty()) return;
>> 
>> I think you've got it!
>> I can't reproduce the issue with your fix - both the reproducer and the monkey tester work correctly.
>> 
>> You don't need to keep the commented out code, please remove it.
>
> Yes this is what caused the issue and the disclosure nodes to appear randomly
> I will remove it

Yes this is what caused the issue and the disclosure nodes to appear randomly
I will remove it

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1838#discussion_r2237961415