commit 9fa2df1f2b726ed751aba9d670df2e82a730ddb0
author jlabanca@google.com <jlabanca@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Sat Aug 02 02:00:27 2008 +0000
committer jlabanca@google.com <jlabanca@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Sat Aug 02 02:00:27 2008 +0000
tree 7c65a52867306fbbae5c3ae9e1c9d374c26ca49e
parent 940d232ae9ddb174fe9a18706f57dc85fa085626

Fixes a bug in Tree where a null pointer exception can occur if a click event propagates to the root TreeItem, which can happen in IE6.  We now check that the target item is not the root TreeItem.

Patch by: jlabanca
Review by: TBR (rajeev)
 


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

user/src/com/google/gwt/user/client/ui/Tree.java

diff --git a/user/src/com/google/gwt/user/client/ui/Tree.java b/user/src/com/google/gwt/user/client/ui/Tree.java
index 334f166..7d92b6d 100644
--- a/user/src/com/google/gwt/user/client/ui/Tree.java
+++ b/user/src/com/google/gwt/user/client/ui/Tree.java
@@ -797,7 +797,7 @@
     collectElementChain(chain, getElement(), hElem);
 
     TreeItem item = findItemByChain(chain, 0, root);
-    if (item != null) {
+    if (item != null && item != root) {
       if (item.getChildCount() > 0
           && DOM.isOrHasChild(item.getImageElement(), hElem)) {
         item.setState(!item.getState(), true);
@@ -993,8 +993,6 @@
 
   /**
    * Move the tree focus to the specified selected item.
-   * 
-   * @param
    */
   private void moveFocus() {
     HasFocus focusableWidget = curSelection.getFocusableWidget();