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

Unified Diff: LayoutTests/fast/css/overflow-rtl-border-after.html

Issue 18720003: Correct overflow propagation in BTT and RTL writing-modes (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 7 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: LayoutTests/fast/css/overflow-rtl-border-after.html
diff --git a/LayoutTests/fast/css/overflow-rtl-border-after.html b/LayoutTests/fast/css/overflow-rtl-border-after.html
new file mode 100644
index 0000000000000000000000000000000000000000..579bb18d63f2df94819de50f5794b4c000754541
--- /dev/null
+++ b/LayoutTests/fast/css/overflow-rtl-border-after.html
@@ -0,0 +1,33 @@
+<!DOCTYPE html>
+<html>
+ <head>
+ <title>Border-after in vertical-rl mode, non-visible overflow</title>
+ <style>
+ body { color:black; background:white; }
+ #container {
+ -webkit-writing-mode: vertical-rl;
+ overflow:hidden;
+ -webkit-logical-height:100px;
+ -webkit-border-after:150px solid white;
Julien - ping for review 2013/07/15 23:50:33 I would rather have the test use physical coordina
mstensho (USE GERRIT) 2013/07/17 13:39:36 Done.
+ background:red;
+ }
+ #inner { -webkit-logical-height:200px; background:white; }
+ </style>
+ <script>
+ if (window.testRunner)
+ testRunner.dumpAsText();
+ </script>
+ </head>
+ <body>
+ <p>The word 'HEST' should be seen below, and there should be no red.</p>
+ <div id="container">
+ <div id="inner">HEST</div>
+ </div>
+ <script>
+ if (window.testRunner) {
+ var logicalHeight = container.scrollWidth;
+ document.body.innerHTML = logicalHeight == 200 ? "PASS" : "FAIL - logicalHeight was " + logicalHeight;
Julien - ping for review 2013/07/15 23:50:33 I think this should be a check-layout.js test. Thi
mstensho (USE GERRIT) 2013/07/17 13:39:36 Done. And thanks for the tip.
+ }
+ </script>
+ </body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698