From be8461d94d8bdb635553754d6f2a541678ef4a92 Mon Sep 17 00:00:00 2001
From: Henri Sara <henri.sara@gmail.com>
Date: Tue, 14 Feb 2017 09:32:54 +0200
Subject: [PATCH] Fix TreeTablePartialUpdatesTest for Chrome 56+ (#8554)

---
 .../components/treetable/TreeTablePartialUpdatesTest.java | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/uitest/src/test/java/com/vaadin/tests/components/treetable/TreeTablePartialUpdatesTest.java b/uitest/src/test/java/com/vaadin/tests/components/treetable/TreeTablePartialUpdatesTest.java
index 9620c01395..767e0270cd 100644
--- a/uitest/src/test/java/com/vaadin/tests/components/treetable/TreeTablePartialUpdatesTest.java
+++ b/uitest/src/test/java/com/vaadin/tests/components/treetable/TreeTablePartialUpdatesTest.java
@@ -28,6 +28,7 @@ import org.openqa.selenium.support.ui.ExpectedConditions;
 import com.vaadin.testbench.By;
 import com.vaadin.testbench.commands.TestBenchElementCommands;
 import com.vaadin.testbench.elements.TreeTableElement;
+import com.vaadin.testbench.parallel.BrowserUtil;
 import com.vaadin.tests.tb3.MultiBrowserTest;
 
 /**
@@ -67,7 +68,12 @@ public class TreeTablePartialUpdatesTest extends MultiBrowserTest {
 
         // scroll far enough down to drop the first row from the cache
         // but not far enough to reach the last row
-        scrollable.scroll(1692);
+        if (BrowserUtil.isChrome(getDesiredCapabilities())) {
+            // Chrome 56 requires different scroll position
+            scrollable.scroll(846);
+        } else {
+            scrollable.scroll(1692);
+        }
 
         // wait for the scrollposition element to disappear
         waitUntilNot(ExpectedConditions.visibilityOfElementLocated(
-- 
2.39.5