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

Side by Side Diff: LayoutTests/paint/selection/text-selection-newline-span-across-line.html

Issue 1350973003: Invalidate line break selection rects when selected. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Don't need redundant early-out for selection rect. 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 unified diff | Download patch
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <html> 2 <html>
3 <head> 3 <head>
4 <script src="../../resources/ahem.js"></script> 4 <script src="../../resources/ahem.js"></script>
5 <script src="../../resources/run-after-layout-and-paint.js"></script> 5 <script src="../../resources/run-after-layout-and-paint.js"></script>
6 <script src="resources/selection.js"></script> 6 <script src="resources/selection.js"></script>
7 <style> 7 <style>
8 div { 8 div {
9 font-family: Ahem; 9 font-family: Ahem;
10 width: 4em; 10 width: 4em;
11 } 11 }
12 </style> 12 </style>
13 </head> 13 </head>
14 <body onload="selectRange(container, 0, container, 3);"> 14 <body onload="selectRangeAfterLayoutAndPaint(container, 0, container, 3);">
15 <!-- Two lines in one block with a span on the first line spanning lines. 15 <!-- Two lines in one block with a span on the first line spanning lines.
16 Should wrap and include space representing a newline after the first 16 Should wrap and include space representing a newline after the first
17 line. --> 17 line. -->
18 <div id="container"> 18 <div id="container">
19 x <span>y z</span> a 19 x <span>y z</span> a
20 </div> 20 </div>
21 </body> 21 </body>
OLDNEW
« no previous file with comments | « LayoutTests/paint/selection/text-selection-newline-span.html ('k') | Source/core/layout/LayoutBR.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698