[Rev 03] RFR: 8236259: MemoryLeak in ProgressIndicator

[Jeanette Winzenburg]

On Thu, 5 Mar 2020 21:47:35 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> the api is in a related issue (should be noted in the issue above). The use-cases mentioned in the isssue: it's a do-once scenario with the skin (and its subclasses) as the only user: multiple listeners can be registered, typically by a skin and/or its subclasses, each can be pre/postpended (or replaced) in the chain for that property and all are unregistered at dispose time. So I think it's really bad to use it in another collaborator like the indicator, particularly if that is re-created often and each of the instances need to remove the listener.
>
> I think this will be fine. The `text` node is private to the `DeterminateIndicator` inner class and not exposed, so there would never be another listener.

ahh yeah, you are right - somehow missed that text is private to the private indicator

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

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