From: Matti Tahvonen <matti.tahvonen@itmill.com>
Date: Mon, 23 Feb 2009 10:31:20 +0000 (+0000)
Subject: server side sanity check for #2607
X-Git-Tag: 6.7.0.beta1~3089
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=3a7e792938264c60e069deec3d28d0f9d9fce1bd;p=vaadin-framework.git

server side sanity check for #2607

svn changeset:6936/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/ui/Table.java b/src/com/itmill/toolkit/ui/Table.java
index 2b528780ca..6588fe1fe5 100644
--- a/src/com/itmill/toolkit/ui/Table.java
+++ b/src/com/itmill/toolkit/ui/Table.java
@@ -1935,6 +1935,14 @@ public class Table extends AbstractSelect implements Action.Container,
         int end = cells[0].length;
         if (reqRowsToPaint != -1) {
             end = start + reqRowsToPaint;
+            // Safari 3.1 sends invalid request sometimes. Cannot reproduce this
+            // with recent webkit. check for valid value and fallback to send
+            // rows to the end of buffer.
+            // TODO replace this with a client side workaround (this causes an
+            // empty screen). See #2607
+            if (end > cells[0].length) {
+                end = cells[0].length;
+            }
         }
         // sanity check
         if (lastToBeRenderedInClient != -1 && lastToBeRenderedInClient < end) {