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: third_party/WebKit/LayoutTests/fast/dom/horizontal-scrollbar-when-dir-change.html

Issue 2165763002: Fix layout tests that would fail if ScrollAnchoring was promoted to experimental (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Undo promoting ScrollAnchoring to test Created 4 years, 5 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: third_party/WebKit/LayoutTests/fast/dom/horizontal-scrollbar-when-dir-change.html
diff --git a/third_party/WebKit/LayoutTests/fast/dom/horizontal-scrollbar-when-dir-change.html b/third_party/WebKit/LayoutTests/fast/dom/horizontal-scrollbar-when-dir-change.html
deleted file mode 100644
index 6816e8cc7fd83742a34634be3f303f33181b6f55..0000000000000000000000000000000000000000
--- a/third_party/WebKit/LayoutTests/fast/dom/horizontal-scrollbar-when-dir-change.html
+++ /dev/null
@@ -1,52 +0,0 @@
-<html>
-<head>
-<script>
-function log(str)
-{
- var li = document.createElement("li");
- li.appendChild(document.createTextNode(str));
- var console = document.getElementById("console");
- console.appendChild(li);
-}
-
-function assertEqual(message, actual, expected)
-{
- if (actual != expected)
- log("\n" + message + ": Failure, actual: " + actual + "; expected: " + expected);
- else
- log("\n" + message + ": Success");
-}
-
-function changeDir(new_dir)
-{
- document.body.dir = new_dir;
- return false;
-}
-
-onload = function()
-{
- if (window.testRunner)
- testRunner.dumpAsText();
-
- assertEqual("original scroll x for RTL page ", window.scrollX, 0);
- window.scrollTo(-100, 0);
- assertEqual("horizontal scroll x for RTL page ", window.scrollX, -100);
-
- document.body.dir = "ltr";
ymalik 2016/07/22 16:53:23 @skobes, Changing rtl to ltr causes a layout and w
skobes 2016/07/25 20:46:38 This test is intended to check the behavior when d
ymalik 2016/07/26 21:03:50 Thanks. Done.
- assertEqual("original scroll x for LTR page ", window.scrollX, 0);
- window.scrollTo(100, 0);
- assertEqual("horizontal scroll x for LTR page ", window.scrollX, 100);
-
- document.getElementById("div").style.display = "none";
-}
-</script>
-</head>
-<body dir="rtl">
-
-<div style="width: 4000px; border: 1px solid red;" id="div">
-Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nunc et nisl tellus. Vivamus aliquam elit non arcu scelerisque eget dapibus nunc vulputate. Maecenas scelerisque egestas purus, id bibendum magna venenatis vel. Vestibulum bibendum volutpat aliquam. Cum sociis natoque penatibus et magnis dis parturient montes, nascetur ridiculus mus. Vestibulum eget risus sodales mi tristique dignissim et sed lectus. Cras convallis, nibh ut dapibus gravida, enim mauris molestie elit, eget fringilla augue est nec est. Curabitur eu quam non purus sodales pulvinar. Donec ultrices dui eget dolor lobortis in porta quam rutrum. Nulla risus lacus, adipiscing quis ullamcorper vestibulum, condimentum sed lacus. Sed in nunc dolor, quis egestas turpis. Integer arcu tellus, posuere sit amet pellentesque a, molestie at mi. Nulla blandit, mi et euismod imperdiet, augue arcu ornare felis, nec viverra lacus ligula ut metus. Proin ut leo ante, at consectetur sem. Nullam non nunc ligula. Proin sed ante sem, id rutrum purus.
-</div>
-<ul id="console"></ul>
-<br>
-</body>
-</html>

Powered by Google App Engine
This is Rietveld 408576698