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

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: tdresser cr feedback 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..368b56b1d121d13000bbf58a2606cff95ad83853 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 document.body is potentially scrollable (with overflow:hidden
+ // the box can still be scrolled programmatically).
+ 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