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

Unified Diff: third_party/WebKit/LayoutTests/paint/text/text-match-highlights-big-line-height.html

Issue 2802543002: Update window.internals.addTextMatchMarker() to take enum instead of bool (Closed)
Patch Set: Fix one more test, add k prefix Created 3 years, 8 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/paint/text/text-match-highlights-big-line-height.html
diff --git a/third_party/WebKit/LayoutTests/paint/text/text-match-highlights-big-line-height.html b/third_party/WebKit/LayoutTests/paint/text/text-match-highlights-big-line-height.html
index 45d6f255770fbd7a5cfbf0d282cfc8e90b8860f4..cf8c6b2648821b9d1f513ad43b54551b6241ccad 100644
--- a/third_party/WebKit/LayoutTests/paint/text/text-match-highlights-big-line-height.html
+++ b/third_party/WebKit/LayoutTests/paint/text/text-match-highlights-big-line-height.html
@@ -18,8 +18,8 @@ function highlightRange(element, start, end, active) {
if (window.internals) {
onload = function() {
- highlightRange(document.getElementById('b1').nextSibling, 0, 3, false);
- highlightRange(document.getElementById('b2').nextSibling, 0, 3, true);
+ highlightRange(document.getElementById('b1').nextSibling, 0, 3, 'kInactive');
+ highlightRange(document.getElementById('b2').nextSibling, 0, 3, 'kActive');
internals.setMarkedTextMatchesAreHighlighted(document, true);
};
}

Powered by Google App Engine
This is Rietveld 408576698