Refactor selection in CellList to allow targeted overriding of selection behavior. This is the absolute smallest change possible to allow custom selection management without duplication of logic.

Review by: jlabanca@google.com

git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@8895 8db76d5a-ed1c-0410-87a9-c151d255dfc7

user/src/com/google/gwt/user/cellview/client/CellList.java

diff --git a/user/src/com/google/gwt/user/cellview/client/CellList.java b/user/src/com/google/gwt/user/cellview/client/CellList.java
index 7eaaa66..df4c0fb 100644
--- a/user/src/com/google/gwt/user/cellview/client/CellList.java
+++ b/user/src/com/google/gwt/user/cellview/client/CellList.java
@@ -257,6 +257,13 @@
     return cell.dependsOnSelection();
   }
 
+  protected void doSelection(Event event, T value, int indexOnPage) {
+    SelectionModel<? super T> selectionModel = getSelectionModel();
+    if (selectionModel != null) {
+      selectionModel.setSelected(value, true);
+    }
+  }
+
   /**
    * Fire an event to the cell.
    *
@@ -349,9 +356,8 @@
         return;
       }
       T value = getDisplayedItem(indexOnPage);
-      SelectionModel<? super T> selectionModel = getSelectionModel();
-      if (selectionModel != null && isMouseDown && !cell.handlesSelection()) {
-        selectionModel.setSelected(value, true);
+      if (isMouseDown && !cell.handlesSelection()) {
+        doSelection(event, value, indexOnPage);
       }
 
       // Focus on the cell.