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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: LayoutTests/fast/multicol/caret-range-outside-columns.html

Issue 978603003: [New Multicolumn] Make positionForPoint() work. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: code review Created 5 years, 9 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
« no previous file with comments | « LayoutTests/TestExpectations ('k') | LayoutTests/fast/multicol/caret-range-outside-columns-expected.txt » ('j') | LayoutTests/fast/pagination/caret-range-outside-paged-y.html » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/fast/multicol/caret-range-outside-columns.html
diff --git a/LayoutTests/fast/multicol/caret-range-outside-columns.html b/LayoutTests/fast/multicol/caret-range-outside-columns.html
new file mode 100644
index 0000000000000000000000000000000000000000..1dd360c095892fb4d78be574a345183e7b62494c
--- /dev/null
+++ b/LayoutTests/fast/multicol/caret-range-outside-columns.html
@@ -0,0 +1,69 @@
+<!DOCTYPE html>
+<style>
+ body { margin:0; }
+ #mc { -webkit-columns:3; -webkit-column-gap:20px; width:280px; margin:20px 100px; font:20px/1 Ahem; }
Julien - ping for review 2015/03/12 14:13:00 We don't have an explicit coding style for tests b
mstensho (USE GERRIT) 2015/03/13 09:34:19 Done.
+
+ /* We don't want the log to get too close to the multicol container, or the tests might fail. */
Julien - ping for review 2015/03/12 14:13:00 I don't think I understand that: the log is absolu
mstensho (USE GERRIT) 2015/03/13 09:34:19 OK, I'll position it at top:101px instead. The low
+ #log { position:absolute; top:50%; }
+</style>
+<body>
Julien - ping for review 2015/03/12 14:13:00 No need for <body>
mstensho (USE GERRIT) 2015/03/13 09:34:19 Done.
+ <div id="log">
+ <p id="description"></p>
+ <div id="console"></div>
+ </div>
+ <div id="mc">xxx xxx xxx xxx xxx xxx xxx xxx xxx</div>
+ <script src="../../resources/js-test.js"></script>
+ <script>
+ description("Test that hitting areas outside columns takes us to the right text content inside columns.");
+
+ // Above first column
Julien - ping for review 2015/03/12 14:13:00 This looks like it should be dumped into the outpu
mstensho (USE GERRIT) 2015/03/13 09:34:19 Done.
+ shouldBe("document.caretRangeFromPoint(50,0).startOffset", "0");
+ shouldBe("document.caretRangeFromPoint(150,0).startOffset", "0");
+ shouldBe("document.caretRangeFromPoint(189,0).startOffset", "0");
+
+ // Above second column
+ shouldBe("document.caretRangeFromPoint(190,0).startOffset", "12");
+ shouldBe("document.caretRangeFromPoint(250,0).startOffset", "12");
+ shouldBe("document.caretRangeFromPoint(289,0).startOffset", "12");
+
+ // Above third column
+ shouldBe("document.caretRangeFromPoint(290,0).startOffset", "24");
+ shouldBe("document.caretRangeFromPoint(350,0).startOffset", "24");
+ shouldBe("document.caretRangeFromPoint(450,0).startOffset", "24");
+
+ // Below first column
+ shouldBe("document.caretRangeFromPoint(50,100).startOffset", "12");
+ shouldBe("document.caretRangeFromPoint(150,100).startOffset", "12");
+ shouldBe("document.caretRangeFromPoint(189,100).startOffset", "12");
+
+ // Below second column
+ shouldBe("document.caretRangeFromPoint(190,100).startOffset", "24");
+ shouldBe("document.caretRangeFromPoint(250,100).startOffset", "24");
+ shouldBe("document.caretRangeFromPoint(289,100).startOffset", "24");
+
+ // To the left of the first column
+ shouldBe("document.caretRangeFromPoint(0,10).startOffset", "0");
+ shouldBe("document.caretRangeFromPoint(0,30).startOffset", "0");
+ shouldBe("document.caretRangeFromPoint(0,50).startOffset", "4");
+ shouldBe("document.caretRangeFromPoint(0,70).startOffset", "8");
+ shouldBe("document.caretRangeFromPoint(0,90).startOffset", "12");
+
+ // To the right of the third column
+ shouldBe("document.caretRangeFromPoint(400,10).startOffset", "24");
+ shouldBe("document.caretRangeFromPoint(400,30).startOffset", "27");
+ shouldBe("document.caretRangeFromPoint(400,50).startOffset", "31");
+ shouldBe("document.caretRangeFromPoint(400,70).startOffset", "35");
+
+ // In the gap between the first and second columns
+ shouldBe("document.caretRangeFromPoint(180,20).startOffset", "3");
+ shouldBe("document.caretRangeFromPoint(180,40).startOffset", "7");
+ shouldBe("document.caretRangeFromPoint(180,60).startOffset", "11");
+ shouldBe("document.caretRangeFromPoint(180,69).startOffset", "11");
+
+ // In the gap between the second and third columns
+ shouldBe("document.caretRangeFromPoint(280,20).startOffset", "15");
+ shouldBe("document.caretRangeFromPoint(280,40).startOffset", "19");
+ shouldBe("document.caretRangeFromPoint(280,60).startOffset", "23");
+ shouldBe("document.caretRangeFromPoint(280,69).startOffset", "23");
+ </script>
+</body>
« no previous file with comments | « LayoutTests/TestExpectations ('k') | LayoutTests/fast/multicol/caret-range-outside-columns-expected.txt » ('j') | LayoutTests/fast/pagination/caret-range-outside-paged-y.html » ('J')

Powered by Google App Engine
This is Rietveld 408576698