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

Unified Diff: LayoutTests/fast/dom/window-scroll-scaling.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/dom/window-scroll-scaling.html
diff --git a/LayoutTests/fast/dom/window-scroll-scaling.html b/LayoutTests/fast/dom/window-scroll-scaling.html
index 067304012d2a3efcabc7558eeff73bb7388232d0..d0da529934bd012b7eaf03cbb59e9b6cb33c75b8 100644
--- a/LayoutTests/fast/dom/window-scroll-scaling.html
+++ b/LayoutTests/fast/dom/window-scroll-scaling.html
@@ -19,8 +19,8 @@
body.style["width"] = window.innerWidth + 100 + "px";
body.style["height"] = window.innerHeight + 100 + "px";
tdresser 2015/05/19 13:53:11 Is this correct? Should we be setting the size of
Rick Byers 2015/05/20 15:10:24 Either would work, but since this test is explicit
- var originalScrollWidth = body.scrollWidth;
- var originalScrollHeight = body.scrollHeight;
+ var originalScrollWidth = document.scrollingElement.scrollWidth;
+ var originalScrollHeight = document.scrollingElement.scrollHeight;
var scale = 1.1;
if (window.eventSender)
@@ -28,11 +28,11 @@
// As we have increased the scale factor, the innerWidth will be less
// as fewer CSS pixels will be rendered in the same viewport, so
- // body.scrollWidth (scrollHeight) will still be measuring the document
+ // scrollingElement.scrollWidth (scrollHeight) will still be measuring the document
tdresser 2015/05/19 13:53:11 nit: scrollingElement->document.scrollingElement
// content width (height).
- shouldBe("body.scrollWidth", "originalScrollWidth");
- shouldBe("body.scrollHeight", "originalScrollHeight");
+ shouldBe("document.scrollingElement.scrollWidth", "originalScrollWidth");
+ shouldBe("document.scrollingElement.scrollHeight", "originalScrollHeight");
</script>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698