ADDITIONAL SYSTEM INFORMATION :
Windows 10 64-bit

A DESCRIPTION OF THE PROBLEM :
Changing the ListView's data model using setItems prevents ListView from being garbage collected because SelectedItemsReadOnlyObservableList adds a non-weak listener to the data model (ObservableList)

REGRESSION : Last worked in version 8u192

STEPS TO FOLLOW TO REPRODUCE THE PROBLEM :
- create a ListView
- change its data model by calling setItems( ObservableList<T> )
- clear all references to the ListView but keep the data model
- ListView cannot be garbage collected. This worked in earlier releases (JavaFX 8)


EXPECTED VERSUS ACTUAL BEHAVIOR :
EXPECTED -
In earlier releases the ListView could be garbage collected even when the data model was kept
ACTUAL -
see UnitTest: test succeeded with JavaFX 8, but fails with JavaFX 11.0.2 and 12.0.1

---------- BEGIN SOURCE ----------
package listview;

import static org.junit.Assert.fail;

import java.lang.ref.WeakReference;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.TimeUnit;

import org.junit.Assert;
import org.junit.Before;
import org.junit.BeforeClass;
import org.junit.Test;

import javafx.application.Application;
import javafx.application.Platform;
import javafx.collections.FXCollections;
import javafx.collections.ObservableList;
import javafx.scene.control.ListView;
import javafx.stage.Stage;

public class ListViewTest {
static CountDownLatch startupLatch;

private WeakReference<ListView<?>> listViewRef;

public static class TestApp extends Application {

@Override
public void start(Stage stage) throws Exception {
startupLatch.countDown();
}
}

private static ObservableList<String> items = FXCollections.observableArrayList();

public ListViewTest() {

}

@BeforeClass
public static void initJavaFX() {
startupLatch = new CountDownLatch(1);
new Thread(() -> Application.launch(TestApp.class, (String[]) null)).start();
try {
if (!startupLatch.await(15, TimeUnit.SECONDS)) {
fail("Timeout waiting for FX runtime to start");
}
} catch (InterruptedException ex) {
fail("Unexpected exception: " + ex);
}
}

@Before
public void initListView() throws Exception {
CountDownLatch latch = new CountDownLatch(1);

Platform.runLater(() -> {
listViewRef = new WeakReference<>(new ListView<>(items));

latch.countDown();

});

if (!latch.await(15, TimeUnit.SECONDS)) {
fail("Timeout waiting for FX listview initialization");
}

}
@Test
public void testGarbageCollect() {
for (int i = 0; i < 10; i++) {
System.gc();
System.runFinalization();

if (listViewRef.get() == null) {
break;
}

try {
Thread.sleep(500);
} catch (Exception ex) {

}
}
Assert.assertTrue("Could not garbage collect listview", listViewRef.get() == null);
}
}
---------- END SOURCE ----------

CUSTOMER SUBMITTED WORKAROUND :
clear data model by calling setItems(null) to dispose ListView

FREQUENCY : always