Integrated: 8273071: SeparatorSkin: must remove child on dispose
Jeanette Winzenburg
fastegal at openjdk.java.net
Fri Sep 3 10:10:35 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
This pull request has now been integrated.
Changeset: 2267b115
Author: Jeanette Winzenburg <fastegal at openjdk.org>
URL: https://git.openjdk.java.net/jfx/commit/2267b115f907e7c9c447691569e48cf0b2b95029
Stats: 14 lines in 2 files changed: 7 ins; 7 del; 0 mod
8273071: SeparatorSkin: must remove child on dispose
Reviewed-by: arapte
-------------
PR: https://git.openjdk.java.net/jfx/pull/616
More information about the openjfx-dev
mailing list