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