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

Unified Diff: LayoutTests/fast/events/scroll-in-scaled-page-with-overflow-hidden.html

Issue 1133693002: Update most LayoutTests to be agnostic to scrollTopLeftInterop mode (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix a couple more tests Created 5 years, 7 months 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: LayoutTests/fast/events/scroll-in-scaled-page-with-overflow-hidden.html
diff --git a/LayoutTests/fast/events/scroll-in-scaled-page-with-overflow-hidden.html b/LayoutTests/fast/events/scroll-in-scaled-page-with-overflow-hidden.html
index 3266828409c002fb015e1c385cef651705d44048..b8ae5b8636c3c56098916dd75675d6c7fc57b8c0 100644
--- a/LayoutTests/fast/events/scroll-in-scaled-page-with-overflow-hidden.html
+++ b/LayoutTests/fast/events/scroll-in-scaled-page-with-overflow-hidden.html
@@ -26,13 +26,16 @@
if (window.eventSender)
window.eventSender.setPageScaleFactor(2, 0, 0);
- shouldBe("window.document.body.scrollTop", "0");
+ // Note that with ScrollTopLeftInterop enabled, document.scrollingElement is
+ // null because body is potentially scrollable (with overflow:hidden the
+ // box can still be scrolled progamatically).
tdresser 2015/05/19 13:53:11 progamatically -> programmatically body -> docume
Rick Byers 2015/05/20 15:10:24 Done.
+ shouldBe("window.scrollY", "0");
if (window.eventSender) {
eventSender.mouseMoveTo(100, 100);
eventSender.mouseScrollBy(0, -5);
}
- shouldBe("window.document.body.scrollTop", "200");
+ shouldBe("window.scrollY", "200");
})();
</script>

Powered by Google App Engine
This is Rietveld 408576698