RFR: 8273071: SeparatorSkin: must remove child on dispose
Ambarish Rapte
arapte at openjdk.java.net
Fri Sep 3 09:59:38 UTC 2021
On Thu, 2 Sep 2021 12:45:02 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:
> minor skin cleanup issue: SeparatorSkin didn't remove the line it added to the control's children
>
> fix is to override dispose and include the removal
> for testing: removed the exclusion of SeparatorSkin from memoryLeakTest - doing so lets it fail/pass before/after this fix
looks good to me.
-------------
Marked as reviewed by arapte (Reviewer).
PR: https://git.openjdk.java.net/jfx/pull/616
More information about the openjfx-dev
mailing list