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

Unified Diff: LayoutTests/fast/scroll-behavior/main-frame-scrollTop.html

Issue 782793002: Update Element API for CSSOM smooth scrolling to match the spec (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Address review comments Created 6 years 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/scroll-behavior/main-frame-scrollTop.html
diff --git a/LayoutTests/fast/scroll-behavior/main-frame-scrollTop.html b/LayoutTests/fast/scroll-behavior/main-frame-scrollTop.html
index c94704322d1f2334319ac3402841cd2f51fa83f2..7445e801a9066cb0acf4872dcd273ce94c6f4ef5 100644
--- a/LayoutTests/fast/scroll-behavior/main-frame-scrollTop.html
+++ b/LayoutTests/fast/scroll-behavior/main-frame-scrollTop.html
@@ -20,26 +20,16 @@
}
function jsScroll(testCase) {
- if (testCase.js) {
- document.documentElement.scrollTop = {y: testCase.y, behavior: testCase.js};
- } else {
- document.documentElement.scrollTop = testCase.y;
- }
+ document.documentElement.scrollTop = testCase.y;
}
const testScrolls = [
- {js: "instant", css: "auto", x: 0, y: 2},
- {js: "instant", css: "smooth", x: 0, y: 3},
- {js: "auto", css: "auto", x: 0, y: 4},
- {js: "", css: "auto", x: 0, y: 5},
- {js: "smooth", css: "auto", waitForEnd: true, x: 0, y: 15},
- {js: "smooth", css: "smooth", waitForEnd: true, x: 0, y: 25},
- {js: "auto", css: "smooth", waitForEnd: true, x: 0, y: 35},
- {js: "", css: "smooth", waitForEnd: true, x: 0, y: 45},
- {js: "smooth", css: "auto", waitForEnd: false, x: 0, y: 4100},
- {js: "smooth", css: "smooth", waitForEnd: false, x: 0, y: 20},
- {js: "auto", css: "smooth", waitForEnd: false, x: 0, y: 4000},
- {js: "", css: "smooth", waitForEnd: false, x: 0, y: 5},
+ {css: "auto", x: 0, y: 2},
+ {css: "auto", x: 0, y: 4},
+ {css: "smooth", waitForEnd: true, x: 0, y: 25},
+ {css: "smooth", waitForEnd: true, x: 0, y: 45},
+ {css: "smooth", waitForEnd: false, x: 0, y: 4100},
+ {css: "smooth", waitForEnd: false, x: 0, y: 20},
];
function doTest()

Powered by Google App Engine
This is Rietveld 408576698