On Tue, 10 Mar 2026 18:26:42 GMT, Andy Goryachev <[email protected]> wrote:
>> Ziad El Midaoui has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Added Test > > tests/system/src/test/java/test/robot/javafx/scene/treeview/TreeViewInitialFocusTest.java > line 1: > >> 1: package test.robot.javafx.scene.treeview; > > needs the copyright header Added, thanks > tests/system/src/test/java/test/robot/javafx/scene/treeview/TreeViewInitialFocusTest.java > line 77: > >> 75: AtomicReference<TreeItem<String>> focusedItem = new >> AtomicReference<>(); >> 76: AtomicInteger focusedCellCount = new AtomicInteger(-1); >> 77: AtomicReference<String> focusedCellText = new >> AtomicReference<>(); > > `focusedCellCount` and `focusedCellText` are not used - were you planning to > use them in this test? Yes, I used them at first to check the focused cell and forgot to remove it ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2095#discussion_r2915284463 PR Review Comment: https://git.openjdk.org/jfx/pull/2095#discussion_r2915283656
