Changeset 212478 in webkit

Timestamp:

Feb 16, 2017 3:34:29 PM (7 years ago)

Author:

Simon Fraser

Message:

REGRESSION (r212197): [ios-simulator Debug WK2] LayoutTest fast/css/target-fragment-match.html is failing
​https://bugs.webkit.org/show_bug.cgi?id=168239

In iOS WK2, resetting the scroll position after history navigation involves a trip
through the UI process. Attempted to de-flake this test by using a nested setTimeout(0).

• fast/css/target-fragment-match.html:

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• fast/css/target-fragment-match.html (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2017-02-16  Simon Fraser  <simon.fraser@apple.com>
+
+        REGRESSION (r212197): [ios-simulator Debug WK2] LayoutTest fast/css/target-fragment-match.html is failing
+        https://bugs.webkit.org/show_bug.cgi?id=168239
+
+        In iOS WK2, resetting the scroll position after history navigation involves a trip
+        through the UI process. Attempted to de-flake this test by using a nested setTimeout(0).
+
+        * fast/css/target-fragment-match.html:
+
 2017-02-16  Alex Christensen  <achristensen@webkit.org>
 

trunk/LayoutTests/fast/css/target-fragment-match.html

@@ -22 +22 @@
     window.history.back(); // This queues up a navigation, so we need to delay the call to notifyDone.
     if (window.testRunner)
-        setTimeout(function() { testRunner.notifyDone() }, 0);
+        setTimeout(function() {
+            // Second setTimeout is because iOS WK2 updates the scroll position after history navigation via the UI process.
+            setTimeout(function() {
+                testRunner.notifyDone()
+            }, 0);
+        }, 0);
 }
 </script>