From: Artur Signell <artur.signell@itmill.com>
Date: Tue, 23 Feb 2010 12:54:23 +0000 (+0000)
Subject: Fix for #4206 - Setting scroll position doesn't work with Webkit browsers
X-Git-Tag: 6.7.0.beta1~2021
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=512aa1851012bfcf6e2758cf169d88655b99e429;p=vaadin-framework.git

Fix for #4206 - Setting scroll position doesn't work with Webkit browsers

svn changeset:11497/svn branch:6.3
---

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VView.java b/src/com/vaadin/terminal/gwt/client/ui/VView.java
index 838b60a6d8..033d35dab1 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VView.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VView.java
@@ -372,10 +372,6 @@ public class VView extends SimplePanel implements Container, ResizeHandler,
 
         onResize(Window.getClientWidth(), Window.getClientHeight());
 
-        if (BrowserInfo.get().isSafari()) {
-            Util.runWebkitOverflowAutoFix(getElement());
-        }
-
         // finally set scroll position from UIDL
         if (uidl.hasVariable("scrollTop")) {
             scrollable = true;
@@ -387,6 +383,12 @@ public class VView extends SimplePanel implements Container, ResizeHandler,
             scrollable = false;
         }
 
+        // Safari workaround must be run after scrollTop is updated as it sets
+        // scrollTop using a deferred command.
+        if (BrowserInfo.get().isSafari()) {
+            Util.runWebkitOverflowAutoFix(getElement());
+        }
+
         rendering = false;
     }