[Rev 01] RFR: 8241455: Memory leak on replacing selection/focusModel

Jeanette Winzenburg fastegal at openjdk.java.net
Thu Apr 2 09:15:06 UTC 2020


On Wed, 1 Apr 2020 19:34:09 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:

> 
> Will it be good to add a call to `root.getChildren().removeAll()` before adding the `node` to `root` ?

hmm .. don't think that it's necessary to clear out other children: the thing that actually matters is that the control
goes active in the scenegraph and has a skin attached. Can you think of any context where any of the tests might get
whacky if there are more than one nodes in the root?

Actually, that's a pattern (*cough, read that's more or less c&p'ed) I use in other tests, where it sometimes is
allowed and expected that there are more than one.

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

PR: https://git.openjdk.java.net/jfx/pull/148


More information about the openjfx-dev mailing list