Merge

Author: kevinrushforth

modules/javafx.base/src/main/java/javafx/beans/property/adapter/JavaBeanObjectProperty.java

@@ -87,7 +87,7 @@
  * @see javafx.beans.property.ObjectProperty
  * @see JavaBeanObjectPropertyBuilder
  *
- * @param T type of the wrapped {@code Object}
+ * @param <T> type of the wrapped {@code Object}
  * @since JavaFX 2.1
  */
 public final class JavaBeanObjectProperty<T> extends ObjectProperty<T> implements JavaBeanProperty<T> {

modules/javafx.base/src/main/java/javafx/beans/property/adapter/JavaBeanObjectPropertyBuilder.java

@@ -56,7 +56,7 @@
  *
  * @see JavaBeanObjectProperty
  *
- * @param T the type of the wrapped {@code Object}
+ * @param <T> the type of the wrapped {@code Object}
  * @since JavaFX 2.1
  */
 public final class JavaBeanObjectPropertyBuilder<T> {

modules/javafx.base/src/main/java/javafx/beans/property/adapter/ReadOnlyJavaBeanObjectProperty.java

@@ -79,7 +79,7 @@
  * @see javafx.beans.property.ReadOnlyObjectProperty
  * @see ReadOnlyJavaBeanObjectPropertyBuilder
  *
- * @param T the type of the wrapped {@code Object}
+ * @param <T> the type of the wrapped {@code Object}
  * @since JavaFX 2.1
  */
 public final class ReadOnlyJavaBeanObjectProperty<T> extends ReadOnlyObjectPropertyBase<T> implements ReadOnlyJavaBeanProperty<T> {

modules/javafx.base/src/main/java/javafx/beans/property/adapter/ReadOnlyJavaBeanObjectPropertyBuilder.java

@@ -55,7 +55,7 @@
  *
  * @see ReadOnlyJavaBeanObjectProperty
  *
- * @param T the type of the wrapped {@code Object}
+ * @param <T> the type of the wrapped {@code Object}
  * @since JavaFX 2.1
  */
 public final class ReadOnlyJavaBeanObjectPropertyBuilder<T> {

modules/javafx.base/src/main/java/javafx/beans/value/WritableObjectValue.java

@@ -28,8 +28,7 @@
 /**
  * A writable typed value.
  *
- * @param T
- *            The type of the wrapped value
+ * @param <T> the type of the wrapped value
  *
  * @see WritableValue
  *

modules/javafx.controls/src/main/java/com/sun/javafx/scene/control/SelectedItemsReadOnlyObservableList.java

@@ -27,7 +27,6 @@
 import javafx.collections.ListChangeListener;
 import javafx.collections.ObservableList;
 import javafx.collections.ObservableListBase;
-import javafx.collections.WeakListChangeListener;
 
 import java.lang.ref.WeakReference;
 import java.util.ArrayList;
@@ -38,20 +37,7 @@ public abstract class SelectedItemsReadOnlyObservableList<E> extends ObservableL
 
     // This is the actual observable list of selected indices used in the selection model
     private final ObservableList<Integer> selectedIndices;
-
-    private ObservableList<E> itemsList;
-
-    private boolean itemsListChanged = false;
-    private ListChangeListener.Change<? extends E> itemsListChange;
-    private final ListChangeListener itemsListListener = c -> {
-        itemsListChanged = true;
-        itemsListChange = c;
-    };
-    private final WeakListChangeListener weakItemsListListener =
-            new WeakListChangeListener(itemsListListener);
-
     private final Supplier<Integer> modelSizeSupplier;
-
     private final List<WeakReference<E>> itemsRefList;
 
     public SelectedItemsReadOnlyObservableList(ObservableList<Integer> selectedIndices, Supplier<Integer> modelSizeSupplier) {
@@ -100,9 +86,6 @@ public SelectedItemsReadOnlyObservableList(ObservableList<Integer> selectedIndic
                 itemsRefList.add(new WeakReference<>(getModelItem(selectedIndex)));
             }
 
-            itemsListChanged = false;
-            itemsListChange = null;
-
             endChange();
         });
     }
@@ -120,17 +103,6 @@ public int size() {
         return selectedIndices.size();
     }
 
-    // Used by ListView and TableView to allow for improved handling.
-    public void setItemsList(ObservableList<E> itemsList) {
-        if (this.itemsList != null) {
-            this.itemsList.removeListener(weakItemsListListener);
-        }
-        this.itemsList = itemsList;
-        if (itemsList != null) {
-            itemsList.addListener(weakItemsListListener);
-        }
-    }
-
     private E _getModelItem(int index) {
         if (index >= modelSizeSupplier.get()) {
             // attempt to return from the itemsRefList instead

modules/javafx.controls/src/main/java/javafx/scene/control/ListView.java

@@ -31,7 +31,6 @@
 import java.util.List;
 
 import com.sun.javafx.scene.control.Properties;
-import com.sun.javafx.scene.control.SelectedItemsReadOnlyObservableList;
 import com.sun.javafx.scene.control.behavior.ListCellBehavior;
 import javafx.beans.InvalidationListener;
 import javafx.beans.Observable;
@@ -1221,9 +1220,6 @@ public ListViewBitSetSelectionModel(final ListView<T> listView) {
 
             this.listView = listView;
 
-            ((SelectedItemsReadOnlyObservableList)getSelectedItems()).setItemsList(listView.getItems());
-
-
             /*
              * The following two listeners are used in conjunction with
              * SelectionModel.select(T obj) to allow for a developer to select
@@ -1238,7 +1234,6 @@ public ListViewBitSetSelectionModel(final ListView<T> listView) {
                 @Override public void invalidated(Observable observable) {
                     ObservableList<T> oldItems = weakItemsRef.get();
                     weakItemsRef = new WeakReference<>(listView.getItems());
-                    ((SelectedItemsReadOnlyObservableList)getSelectedItems()).setItemsList(listView.getItems());
                     updateItemsObserver(oldItems, listView.getItems());
                 }
             };

modules/javafx.controls/src/main/java/javafx/scene/control/TableView.java

@@ -40,7 +40,6 @@
 import com.sun.javafx.scene.control.Logging;
 import com.sun.javafx.scene.control.Properties;
 import com.sun.javafx.scene.control.SelectedCellsMap;
-import com.sun.javafx.scene.control.SelectedItemsReadOnlyObservableList;
 import com.sun.javafx.scene.control.behavior.TableCellBehavior;
 import com.sun.javafx.scene.control.behavior.TableCellBehaviorBase;
 
@@ -2104,8 +2103,6 @@ public TableViewArrayListSelectionModel(final TableView<S> tableView) {
                     ObservableList<S> oldItems = weakItemsRef.get();
                     weakItemsRef = new WeakReference<>(tableView.getItems());
                     updateItemsObserver(oldItems, tableView.getItems());
-
-                    ((SelectedItemsReadOnlyObservableList)getSelectedItems()).setItemsList(tableView.getItems());
                 }
             };
             this.tableView.itemsProperty().addListener(itemsPropertyListener);
@@ -2142,7 +2139,6 @@ public TableViewArrayListSelectionModel(final TableView<S> tableView) {
             // watching for changes to the items list content
             ObservableList<S> items = getTableView().getItems();
             if (items != null) {
-                ((SelectedItemsReadOnlyObservableList)getSelectedItems()).setItemsList(items);
                 items.addListener(weakItemsContentListener);
             }