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

Unified Diff: third_party/WebKit/LayoutTests/fast/text/selection-with-inline-padding.html

Issue 2546473003: Simplify computation of text selection top/bottom. (Closed)
Patch Set: Add add'l test expectations. Created 4 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/fast/text/selection-with-inline-padding-expected.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/fast/text/selection-with-inline-padding.html
diff --git a/third_party/WebKit/LayoutTests/fast/text/selection-with-inline-padding.html b/third_party/WebKit/LayoutTests/fast/text/selection-with-inline-padding.html
index 56c2a6c19b206dbcb3fd1d9576ce98a6b60b96fb..7992c4b1171f4820a1122d245c253aaefc6757e6 100644
--- a/third_party/WebKit/LayoutTests/fast/text/selection-with-inline-padding.html
+++ b/third_party/WebKit/LayoutTests/fast/text/selection-with-inline-padding.html
@@ -1,27 +1,25 @@
<!DOCTYPE html>
-<html>
- <head>
- <title>Test paddings and selection for inline elements</title>
- <style>
- p > b, pre > b { padding: 10px; }
- </style>
- </head>
- <body>
- <p>
- Lorem ipsum dolor sit amet, consectetur adipiscing elit.<br>
- <b id="test">Sed dictum erat sit amet pharetra pretium.</b><br>
- Nullam a est vitae orci tempus tincidunt nec at dolor.
- </p>
- <p>
- Tests that selections do not include padding in the Y direction for
- inline elements where it should be ignored.
- </p>
- <script>
- var node = document.getElementById('test').firstChild;
- var range = document.createRange();
- range.setStart(node, 5);
- range.setEnd(node, node.length - 5);
- window.getSelection().addRange(range);
- </script>
- </body>
-</html>
+<style>
+p > b {
+ padding: 10px;
+}
+</style>
+<p>
+ Lorem ipsum dolor sit amet, consectetur adipiscing elit.<br>
+ <b id="test">Sed dictum erat sit amet pharetra pretium.</b><br>
+ Nullam a est vitae orci tempus tincidunt nec at dolor.
+</p>
+<p>
+ Tests whether selection includes padding in the Y direction for inline
+ elements. This padding would ideally not be included. http://crbug.com/657325#c13
+</p>
+<script src="../../resources/run-after-layout-and-paint.js"></script>
+<script>
+runAfterLayoutAndPaint(function() {
+ var node = document.getElementById('test').firstChild;
+ var range = document.createRange();
+ range.setStart(node, 5);
+ range.setEnd(node, node.length - 5);
+ window.getSelection().addRange(range);
+}, true);
+</script>
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/fast/text/selection-with-inline-padding-expected.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698