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

Unified Diff: third_party/WebKit/LayoutTests/editing/selection/dont-select-text-overflow-ellipsis-when-wrapping-ltr-mixed.html

Issue 2594983002: Don't paint underline or selections over ellipsis in mixed-flow contexts (Closed)
Patch Set: bug 642454 Created 3 years, 11 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/editing/selection/dont-select-text-overflow-ellipsis-when-wrapping-ltr-mixed.html
diff --git a/third_party/WebKit/LayoutTests/editing/selection/dont-select-text-overflow-ellipsis-when-wrapping-ltr-mixed.html b/third_party/WebKit/LayoutTests/editing/selection/dont-select-text-overflow-ellipsis-when-wrapping-ltr-mixed.html
new file mode 100644
index 0000000000000000000000000000000000000000..16d5af9fa6a6b1ef363ff991e2380d185fde674b
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/editing/selection/dont-select-text-overflow-ellipsis-when-wrapping-ltr-mixed.html
@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<meta charset=utf-8>
+<style>
+dl {
+ max-width: 20em;
+ margin: auto;
+ font-size: 120%;
+ white-space: nowrap;
+}
+dd {
+ overflow: hidden;
+ text-overflow: ellipsis;
+ text-decoration: inherit;
+}
+</style>
+<p>crbug.com/642454: Don't select part of the ellipsis when wrapping selection.</p>
+<dl>
+ <dd id="selection_with_ellipsis">Lorem לורם איפסום הוא כינוי לטקסט חסר משמעות לחלוטין</dd>
+ <dt id="wrapped">Lorem ipsum dolor sit amet</dt>
+</dl>
+<script src="../../resources/run-after-layout-and-paint.js"></script>
+<script>
+runAfterLayoutAndPaint(function() {
+ getSelection().setBaseAndExtent(selection_with_ellipsis.firstChild, 0, wrapped.firstChild, 10);
+ }, true);
+</script>

Powered by Google App Engine
This is Rietveld 408576698