RFR: 8359599: Calling refresh() for all virtualized controls recreates all cells instead of refreshing the cells [v4]

Kevin Rushforth kcr at openjdk.org
Wed Oct 29 17:40:59 UTC 2025


On Mon, 13 Oct 2025 12:55:58 GMT, Marius Hanl <mhanl at openjdk.org> wrote:

>> Marius Hanl has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Improve javadoc
>
> There seems to be an intermediate test failure, maybe a race condition unrelated to this change:
> 
> 
> TaskEventTest > cancelledCalledAfterHandler() FAILED
>     org.opentest4j.AssertionFailedError: expected: <true> but was: <false>
>         at app//org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
>         at app//org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
>         at app//org.junit.jupiter.api.AssertTrue.failNotTrue(AssertTrue.java:63)
>         at app//org.junit.jupiter.api.AssertTrue.assertTrue(AssertTrue.java:36)
>         at app//org.junit.jupiter.api.AssertTrue.assertTrue(AssertTrue.java:31)
>         at app//org.junit.jupiter.api.Assertions.assertTrue(Assertions.java:183)
>         at app//test.javafx.concurrent.TaskEventTest.cancelledCalledAfterHandler(TaskEventTest.java:410)

@Maran23 In case you missed it, this is pending changes from you.

1. I noted some final doc changes in [this comment above](#discussion_r2455868260).
2. I had added an [earlier comment](#issuecomment-3393245504) about adding your perf test to the PR somewhere under `tests/performance`.

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

PR Comment: https://git.openjdk.org/jfx/pull/1830#issuecomment-3462826572


More information about the openjfx-dev mailing list