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

Unified Diff: LayoutTests/fast/scroll-behavior/main-frame-scrollLeft.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-scrollLeft.html
diff --git a/LayoutTests/fast/scroll-behavior/main-frame-scrollLeft.html b/LayoutTests/fast/scroll-behavior/main-frame-scrollLeft.html
index 175593d4130a3025a58f0eeef0b39392f4e9ea2e..68c3f69e8ba5a2f921be0354970e81dbd9cc7b1c 100644
--- a/LayoutTests/fast/scroll-behavior/main-frame-scrollLeft.html
+++ b/LayoutTests/fast/scroll-behavior/main-frame-scrollLeft.html
@@ -20,26 +20,16 @@
}
function jsScroll(testCase) {
- if (testCase.js) {
- document.documentElement.scrollLeft = {x: testCase.x, behavior: testCase.js};
- } else {
- document.documentElement.scrollLeft = testCase.x;
- }
+ document.documentElement.scrollLeft = testCase.x;
}
const testScrolls = [
- {js: "instant", css: "auto", x: 1, y: 0},
- {js: "instant", css: "smooth", x: 2, y: 0},
- {js: "auto", css: "auto", x: 3, y: 0},
- {js: "", css: "auto", x: 4, y: 0},
- {js: "smooth", css: "auto", waitForEnd: true, x: 10, y: 0},
- {js: "smooth", css: "smooth", waitForEnd: true, x: 20, y: 0},
- {js: "auto", css: "smooth", waitForEnd: true, x: 30, y: 0},
- {js: "", css: "smooth", waitForEnd: true, x: 40, y: 0},
- {js: "smooth", css: "auto", waitForEnd: false, x: 4000, y: 0},
- {js: "smooth", css: "smooth", waitForEnd: false, x: 15, y: 0},
- {js: "auto", css: "smooth", waitForEnd: false, x: 4100, y: 0},
- {js: "", css: "smooth", waitForEnd: false, x: 10, y: 0},
+ {css: "auto", x: 1, y: 0},
+ {css: "auto", x: 4, y: 0},
+ {css: "smooth", waitForEnd: true, x: 20, y: 0},
+ {css: "smooth", waitForEnd: true, x: 40, y: 0},
+ {css: "smooth", waitForEnd: false, x: 4000, y: 0},
+ {css: "smooth", waitForEnd: false, x: 10, y: 0},
];
function doTest()

Powered by Google App Engine
This is Rietveld 408576698