The issue is that the toggles is not reliably unselected if an uncontained 
value is set.

The root is ChoiceBoxSelectionModel which doesn't update the index on selecting 
an uncontained item, in particular it
fails to keep the invariant:

     assertEquals(getItems().indexOf(selectedItem), selectedIndex);

The fix here is to override select(item) to guarantee the assert.

Added/removed ignore from tests that failed before and pass after the fix. All 
other tests are passing before and after.

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

Commit messages:
 - 8241999: ChoiceBox: incorrect toggle selected for uncontained

Changes: https://git.openjdk.java.net/jfx/pull/200/files
 Webrev: https://webrevs.openjdk.java.net/jfx/200/webrev.00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8241999
  Stats: 55 lines in 2 files changed: 44 ins; 7 del; 4 mod
  Patch: https://git.openjdk.java.net/jfx/pull/200.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/200/head:pull/200

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

Reply via email to