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

Unified Diff: LayoutTests/fast/repaint/text-match-pre-wrapped-text.html

Issue 1305953003: Find In Page doesn't correctly highlight wrapped text. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: 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/fast/repaint/text-match-pre-wrapped-text.html
diff --git a/LayoutTests/fast/repaint/text-match-transparent-text.html b/LayoutTests/fast/repaint/text-match-pre-wrapped-text.html
similarity index 66%
copy from LayoutTests/fast/repaint/text-match-transparent-text.html
copy to LayoutTests/fast/repaint/text-match-pre-wrapped-text.html
index 6540ce66688a0fb8b7961652d38196e28a6cf2b4..65a73cbfc37a379845c4ffab7d9ddd916bd33c1d 100644
--- a/LayoutTests/fast/repaint/text-match-transparent-text.html
+++ b/LayoutTests/fast/repaint/text-match-pre-wrapped-text.html
@@ -9,12 +9,11 @@
window.internals.setMarkedTextMatchesAreHighlighted(document, true);
}
function highlightTest() {
- highlightRange('1', 2, 5, true);
+ highlightRange('1', 53, 60, true);
}
</script>
<body onload="highlightTest()">
-Test for crbug.com/504712
-<p>Find In Page: The below transparent text "ndM" from findMe should be highlighted as orange and painted as black</p>
-<span id="1" style="color:transparent">findMe</span>
+Test for crbug.com/521047
+<pre id="1" style="max-width:460px; white-space:pre-wrap">Find In Page should correctly highlight wrapped text Find Me </pre>
</body>

Powered by Google App Engine
This is Rietveld 408576698