RFR: 8256283: IndexOutOfBoundsException when sorting a TreeTableView [v2]

[Kevin Rushforth]

On Thu, 28 Jan 2021 11:48:02 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:

>> This particular issue JDK-8256283, is a specific case of IOOBE when, rootItem is not shown, some children including first child are selected, then all children are removed and sort() is invoked. The sort() fails with an IOOBE.
>> This PR only addresses this specific IOOBE.
>> Root cause of this issue is that the selection is not cleared after rootItems children are removed. In addition to this, there are few other scenarios when selection is not updated correctly, which are collected under an umbrella task [JDK-8248217](https://bugs.openjdk.java.net/browse/JDK-8248217). Fix for [JDK-8248217](https://bugs.openjdk.java.net/browse/JDK-8248217) would require good amount refactoring of selection model.
>> 
>> The fix for this issue is to avoid sort() when rootItem.getChildren().isEmpty().
>> Added a unit test with the fix, which fails without fix and passes with fix.
>
> Ambarish Rapte has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review correction

Marked as reviewed by kcr (Lead).

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

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