On Fri, 3 Feb 2023 23:05:43 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> Fixed regression introduced by JDK-8295754, targeting upstream **jfx20 
>> branch**.
>> 
>> The method PaginationSkin.resetIndexes(true) has been moved to the original 
>> position in the constructor, fixing the initialization, while making sure no 
>> properties of the control are touched in the constructor (only in install()).
>> 
>> Added a test case.
>> 
>> Tested with the PaginationDisappear.java and the LeakTest app
>> https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTestApp.java
>
> The changes you just made look good to me, since everything is now internally 
> consistent. I'll retest and then finish my review.

Thank you, @kevinrushforth .

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

PR: https://git.openjdk.org/jfx/pull/1021

Reply via email to