[gwt-contrib] [google-web-toolkit] r8138 committed - Temporary workaround of checking for null values, so that IE doesn't b...

[codesite-noreply]

Revision: 8138
Author: amitman...@google.com
Date: Thu May 13 16:21:15 2010
Log: Temporary workaround of checking for null values, so that IE doesn't  
blow up.
Re-visit.
Patch by: amitmanjhi
Review by: jlabanca (TBR)

http://code.google.com/p/google-web-toolkit/source/detail?r=8138

Modified:
  
/branches/2.1/user/src/com/google/gwt/user/cellview/client/CellListImpl.java

=======================================
---  
/branches/2.1/user/src/com/google/gwt/user/cellview/client/CellListImpl.java	 
Thu May 13 11:08:17 2010
+++  
/branches/2.1/user/src/com/google/gwt/user/cellview/client/CellListImpl.java	 
Thu May 13 16:21:15 2010
@@ -501,11 +501,18 @@
           selectedRows.remove(row);
         }
         if (!dependsOnSelection) {
-          // The cell doesn't depend on selection, so we only need to  
update the
-          // style.
-          setSelected(cellElem, selected);
+          if (cellElem != null) {
+            // TODO: do a better check?
+            // The cell doesn't depend on selection, so we only need to  
update
+            // the style.
+            setSelected(cellElem, selected);
+          }
         }
       }
+      if (cellElem == null) {
+        // TODO: do a better check?
+        break;
+      }
       cellElem = cellElem.getNextSiblingElement();
       row++;
     }

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors