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

Unified Diff: LayoutTests/paint/selection/invalidation-rect-excludes-newline-for-vertical.html

Issue 1355913004: Disable newline selection highlighting for vertical and RTL text. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@474759_invalidate_br_and_horiz_only
Patch Set: Merge changes. Created 5 years, 3 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/paint/selection/invalidation-rect-excludes-newline-for-vertical.html
diff --git a/LayoutTests/paint/selection/invalidation-rect-includes-newline.html b/LayoutTests/paint/selection/invalidation-rect-excludes-newline-for-vertical.html
similarity index 65%
copy from LayoutTests/paint/selection/invalidation-rect-includes-newline.html
copy to LayoutTests/paint/selection/invalidation-rect-excludes-newline-for-vertical.html
index 5fed1477371ba78d2bf051bfb4c9682d8a7330d8..658123ad49a791300d2b5eca3e9376c8aed682b3 100644
--- a/LayoutTests/paint/selection/invalidation-rect-includes-newline.html
+++ b/LayoutTests/paint/selection/invalidation-rect-excludes-newline-for-vertical.html
@@ -3,7 +3,8 @@
<script src="../../fast/repaint/resources/text-based-repaint.js"></script>
<script src="resources/selection.js"></script>
<script>
-// Selects two lines and expects a newline to be represented.
+// Selects two lines and expects a newline to *not* be represented as newline
+// highlight selection is currently only enabled for horizontal text.
function repaintTest() {
selectRange(container, 0, container, 1);
};
@@ -12,7 +13,8 @@ window.onload = runRepaintTest;
<style>
div {
font-family: Ahem;
- width: 2em;
+ height: 2em;
+ -webkit-writing-mode: vertical-lr;
}
</style>
<div id="container">

Powered by Google App Engine
This is Rietveld 408576698