On Tue, 9 May 2023 22:41:41 GMT, Kevin Rushforth <[email protected]> wrote:
>> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains 10 additional >> commits since the last revision: >> >> - removed monkey tester changes >> - Merge remote-tracking branch 'origin/master' into 8307538.refresh >> - whitespace >> - updated test >> - variable tree cell fix >> - fixed size selector >> - list view page >> - fix and test >> - skin menu >> - refresh button > > modules/javafx.controls/src/test/java/test/javafx/scene/control/skin/TreeTableRowSkinTest.java > line 215: > >> 213: int right = 23; >> 214: int bottom = 37; >> 215: int left = 49; > > After the fix and test change, this test now fails for me. I'm not quite seeing why these changes are needed. How are they related to the memory leak? ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1129#discussion_r1189436860
