RFR: 8253086: Optimization of removeAll and retainAll of ObservableListWrapper [v4]

Wed Oct 7 11:33:08 UTC 2020

On Wed, 7 Oct 2020 11:12:43 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:

> BTW, just noticed that Tree/Table/View never had the issue that was fixed with introducing the two-pass approach, can't
> nail why not - they are using the same selectedItems (from MultipleSelectionModelBase), any ideas?

because it's not a live-lookup: TreeView keeps some cache of exposed treeItems - that's to where the getModelItem() in
selectedItems is delegated to - which is updated on receiving a listChange, that is not during the modification.

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

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