Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(47)

Unified Diff: third_party/WebKit/LayoutTests/fast/events/scroll-after-click-on-tab-index.html

Issue 1413493005: Update layout tests to work when smooth scrolling is enabled by default (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: worked on nit Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/events/scroll-after-click-on-tab-index.html
diff --git a/third_party/WebKit/LayoutTests/fast/events/scroll-after-click-on-tab-index.html b/third_party/WebKit/LayoutTests/fast/events/scroll-after-click-on-tab-index.html
index 78058fe3d799e148bc467596fdd6fbc0d22e6d79..98cf4009e60ef5b8d42931cb6f1a31a436606f21 100644
--- a/third_party/WebKit/LayoutTests/fast/events/scroll-after-click-on-tab-index.html
+++ b/third_party/WebKit/LayoutTests/fast/events/scroll-after-click-on-tab-index.html
@@ -1,6 +1,8 @@
<html>
<script src="../../resources/js-test.js"></script>
<script>
+window.jsTestIsAsync = true;
+
function getCenterFor(element)
{
var rect = element.getBoundingClientRect();
@@ -23,14 +25,9 @@ function runTest()
eventSender.mouseDown();
eventSender.mouseUp();
eventSender.keyDown("downArrow");
- setTimeout(verifyScrolling, 15);
-}
-function verifyScrolling()
-{
debug("Scroll position is more than 0");
- shouldBe("document.scrollingElement.scrollTop > 0", "true");
- testRunner.notifyDone();
+ shouldBecomeEqual("document.scrollingElement.scrollTop > 0", "true", finishJSTest);
}
</script>

Powered by Google App Engine
This is Rietveld 408576698