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

Unified Diff: LayoutTests/fast/scroll-behavior/listbox-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/listbox-scrollTop.html
diff --git a/LayoutTests/fast/scroll-behavior/listbox-scrollTop.html b/LayoutTests/fast/scroll-behavior/listbox-scrollTop.html
deleted file mode 100644
index 46e5483e7cbbd18a273e635a88ae9a6eac88c463..0000000000000000000000000000000000000000
--- a/LayoutTests/fast/scroll-behavior/listbox-scrollTop.html
+++ /dev/null
@@ -1,79 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
- <script src="../../resources/testharness.js"></script>
- <script src="../../resources/testharnessreport.js"></script>
- <script src="resources/scroll-behavior-test.js"></script>
- <script type="text/javascript">
- const numItems = 500;
-
- function getEndPosition(testCase, startPosition) {
- var endPosition = {};
- endPosition.x = startPosition.x;
- endPosition.y = testCase.y;
- return endPosition;
- }
-
- function jsScroll(testCase) {
- var element = document.getElementById("listbox");
- if (testCase.js) {
- element.scrollTop = {y: testCase.y, behavior: testCase.js};
- } else {
- element.scrollTop = testCase.y;
- }
- }
-
- var testScrolls = [
- {js: "instant", css: "auto", index: 2},
- {js: "instant", css: "smooth", index: 3},
- {js: "auto", css: "auto", index: 4},
- {js: "", css: "auto", index: 5},
- {js: "smooth", css: "auto", waitForEnd: true, index: 15},
- {js: "smooth", css: "smooth", waitForEnd: true, index: 20},
- {js: "auto", css: "smooth", waitForEnd: true, index: 30},
- {js: "", css: "smooth", waitForEnd: true, index: 35},
- {js: "smooth", css: "auto", waitForEnd: false, index: 410},
- {js: "smooth", css: "smooth", waitForEnd: false, index: 20},
- {js: "auto", css: "smooth", waitForEnd: false, index: 400},
- {js: "", css: "smooth", waitForEnd: false, index: 5},
- ];
-
- function initializeContents(listbox)
- {
- for (var i = 0; i < numItems; i++) {
- var option = document.createElement("option");
- option.appendChild(document.createTextNode(i));
- listbox.appendChild(option);
- }
- }
-
-function doTest()
- {
- var element = document.getElementById("listbox");
- initializeContents(element);
- var itemHeight = Math.floor(listbox.clientHeight / listbox.size);
- var testCases = [];
- for (var i = 0; i < testScrolls.length; i++) {
- // Convert from list box item index to a scroll offset.
- testScrolls[i].y = testScrolls[i].index * itemHeight;
- testScrolls[i].x = 0;
- testCases.push(new ScrollBehaviorTestCase(testScrolls[i]));
- }
-
- var scrollBehaviorTest = new ScrollBehaviorTest(element,
- element,
- testCases,
- getEndPosition,
- jsScroll);
- scrollBehaviorTest.run();
- }
-
- window.addEventListener('load', doTest, false);
- </script>
-</head>
-
-<body>
- <p>Test that setting scrollTop on a listbox works with both scroll behaviors</p>
- <select size="4" id="listbox"></select>
-</body>
-</html>

Powered by Google App Engine
This is Rietveld 408576698