From 3a7e792938264c60e069deec3d28d0f9d9fce1bd Mon Sep 17 00:00:00 2001
From: Matti Tahvonen <matti.tahvonen@itmill.com>
Date: Mon, 23 Feb 2009 10:31:20 +0000
Subject: [PATCH] server side sanity check for #2607

svn changeset:6936/svn branch:trunk
---
 src/com/itmill/toolkit/ui/Table.java | 8 ++++++++
 1 file changed, 8 insertions(+)

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) {
-- 
2.39.5