-
Enhancement
-
Resolution: Unresolved
-
P4
-
None
In the context of JDK-8294809 ListenerHelper, I would like to remove LambdaMultiplePropertyChangeListenerHandler as its functionality will be replaced (and extended) by ListenerHelper.
The reasons to remove LambdaMultiplePropertyChangeListenerHandler:
- it is not symmetric and produces unexpected side effects when removing a listener from a property when multiple listeners have been registered on that property
- its functionality has been replaced by ListenerHelper which not only helps with adding various listeners but also event handlers and event filters
This change will keep existing SkinBase.register*() methods with implementation using ListenerHelper.
This change will make JDK-8265401 obsolete.
TODO: also replace add/remove listener in TextInputControlSkin:330 with ListenerHelper
The reasons to remove LambdaMultiplePropertyChangeListenerHandler:
- it is not symmetric and produces unexpected side effects when removing a listener from a property when multiple listeners have been registered on that property
- its functionality has been replaced by ListenerHelper which not only helps with adding various listeners but also event handlers and event filters
This change will keep existing SkinBase.register*() methods with implementation using ListenerHelper.
This change will make JDK-8265401 obsolete.
TODO: also replace add/remove listener in TextInputControlSkin:330 with ListenerHelper
- relates to
-
JDK-8265401 Rename internal class LambdaMultiplePropertyChangeListenerHandler
- Open
-
JDK-8294809 ListenerHelper for managing and disconnecting listeners
- Resolved