RFR: 8089398: [ChoiceBox, ComboBox] throws NPE on setting value on null selectionModel [v3]

[Jeanette Winzenburg]

On Thu, 8 Jul 2021 10:18:13 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:

>> added.
>
> Hmm ... wondering whether we really want to widen the scope of this issue
> 
> - it started with being focused on NPE on the change of property value, for both Choice/ComboBox
> - turned out combo's skin also has a throwing listener to value
> - review spotted an additional failure candidates in ComboBox' editable property
> - plus a sore spot in combo's skin (one of the locations where list/combo selection is sync'ed, there are others ;)
> 
> the first two are naturally within the original scope, the third is near enough (a property on one of the covered controls) to be included .. the last is arguable, IMO - would tend to not include it here but open a follow-up to then include _all_ sync issues in the skin (probably needs more digging and definitely more testing).
> 
> Thoughts?

hmm ... can't unresolve this (probably because I wasn't involved?) - how to make it part of the review?

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

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