On Thu, 28 Jan 2021 11:48:02 GMT, Ambarish Rapte <ara...@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

Reply via email to