commit 6883fe709050375cfc124b5777315e0518f266a2
author isoos@google.com <isoos@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Thu Feb 02 17:39:36 2012 +0000
committer isoos@google.com <isoos@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Thu Feb 02 17:39:36 2012 +0000
tree 4a80d723383f1a75b0ab8a9f6c589b527e8af19c
parent 72a3b69ddb5a5f0d226ece34eeae265dea786a8e

CellTable performance is very slow on IE8. Internally, AbstractCellTable detects
row hover events, and part of that code will access absolute coordinates of the
row (to safeguard the case where we have items with fixed positions over the
table). Unfortunately, IE 8 is really slow to access these coordinates, because
every DOM modification (including style updates, non-visible or any other DOM
changes) will trigger a full reflow/recalculation.

This fix introduces three flags that can be used to skip parts or all of
CellTable's internal row hover detection and/or style updates. For
backward-compatibility, the new flags shall be set by the user of the
table, otherwise the old behavior remains in effect.

Users are suggested to use these flags at least on IE6/IE7/IE8 and adjust the
related functionality based on their needs. The three flags address the
different use cases outlined in the issue 6167.

Fixes issues: 6167

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

Review by: jlabanca@google.com

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