#8040 test case and fix

svn changeset:22422/svn branch:6.7

1 files changed, 2 insertions, 9 deletions

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
index 553934bf98..d77507084d 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
@@ -4052,15 +4052,8 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
                 reactLastRow = totalRows - 1;
             }
             if (lastRendered < reactFirstRow || firstRendered > reactLastRow) {
-                /*
-                 * #8040 - scroll position is completely changed since the
-                 * latest request, so request a new set of rows.
-                 * 
-                 * TODO: We should probably check whether the fetched rows match
-                 * the current scroll position right when they arrive, so as to
-                 * not waste time rendering a set of rows that will never be
-                 * visible...
-                 */
+                // #8040 - scroll position is completely changed
+                // TODO: should probably be done earlier
                 rowRequestHandler.setReqFirstRow(reactFirstRow);
                 rowRequestHandler.setReqRows(reactLastRow - reactFirstRow + 1);
                 rowRequestHandler.deferRowFetch(1);