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

Unified Diff: LayoutTests/fast/events/resources/drag-and-drop-autoscroll-frame.js

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/resources/drag-and-drop-autoscroll-frame.js
diff --git a/LayoutTests/fast/events/resources/drag-and-drop-autoscroll-frame.js b/LayoutTests/fast/events/resources/drag-and-drop-autoscroll-frame.js
index 95545b371e24ab4ccedeea16b619037f6cd233aa..1f045984228def9ec17fef84c8da58725dce56d4 100644
--- a/LayoutTests/fast/events/resources/drag-and-drop-autoscroll-frame.js
+++ b/LayoutTests/fast/events/resources/drag-and-drop-autoscroll-frame.js
@@ -15,8 +15,8 @@ window.onload = function() {
function moveTo(newState, x, y)
{
state = newState;
- lastScrollLeft = document.body.scrollLeft;
- lastScrollTop = document.body.scrollTop;
+ lastScrollLeft = document.scrollingElement.scrollLeft;
+ lastScrollTop = document.scrollingElement.scrollTop;
eventSender.mouseMoveTo(x, y);
}
@@ -26,23 +26,23 @@ window.onload = function() {
debug('state=' + state);
switch (state) {
case 'NE':
- shouldBeTrue('document.body.scrollLeft > 0');
- shouldBeTrue('!document.body.scrollTop');
+ shouldBeTrue('document.scrollingElement.scrollLeft > 0');
+ shouldBeTrue('!document.scrollingElement.scrollTop');
moveTo('SE', westX, southY);
break;
case 'SE':
- shouldBeTrue('document.body.scrollLeft > 0');
- shouldBeTrue('document.body.scrollTop > 0');
+ shouldBeTrue('document.scrollingElement.scrollLeft > 0');
+ shouldBeTrue('document.scrollingElement.scrollTop > 0');
moveTo('SW', westX, southY);
break;
case 'SW':
- shouldBeTrue('document.body.scrollLeft < lastScrollLeft');
- shouldBeTrue('document.body.scrollTop > 0');
+ shouldBeTrue('document.scrollingElement.scrollLeft < lastScrollLeft');
+ shouldBeTrue('document.scrollingElement.scrollTop > 0');
moveTo('NW', westX, northY);
break;
case 'NW':
- shouldBeTrue('document.body.scrollLeft <= lastScrollLeft');
- shouldBeTrue('document.body.scrollTop < lastScrollTop');
+ shouldBeTrue('document.scrollingElement.scrollLeft <= lastScrollLeft');
+ shouldBeTrue('document.scrollingElement.scrollTop < lastScrollTop');
eventSender.mouseUp();
$('container').outerHTML = '';
if (window.parent !== window)

Powered by Google App Engine
This is Rietveld 408576698