commit 2b64e12336a1893a575776175042b554b81c770b
author jlabanca@google.com <jlabanca@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Mon Aug 08 15:03:35 2011 +0000
committer jlabanca@google.com <jlabanca@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Mon Aug 08 15:03:35 2011 +0000
tree 677a1b248a0bcd6f7ce30a72fbf7f8ce68196dc8
parent c0e8308487471e6d6d2c54913c5831875a4b47f1

Fixes a recently introduced bug in Cell Widgets where clicking on the first row does not update the SelectionModel when bound to selection.  The DefaultKeyboardSelectionHandler has an overly optimistic optimization that ignores the click event if it occurs on the current keyboard selected row.  However, the keyboard selected row is initialized to 0, and updating the keyboard selected row triggers selection, so the optimization blocks selection of the 0th row.  Removing the optimization check fixes the bug.

Review at http://gwt-code-reviews.appspot.com/1509806

Review by: devint@google.com

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