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

Side by Side Diff: third_party/WebKit/LayoutTests/fast/dom/documentmarker-add-adjacent-text.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 unified diff | Download patch
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/fast/dom/documentmarker-set-active.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <html> 1 <html>
2 <script src="../../resources/js-test.js"></script> 2 <script src="../../resources/js-test.js"></script>
3 <body style="min-height: 5000px"> 3 <body style="min-height: 5000px">
4 <span id="elt">text</span> 4 <span id="elt">text</span>
5 5
6 <script> 6 <script>
7 description('Checks that two adjacent TextMatch markers are not being merged.'); 7 description('Checks that two adjacent TextMatch markers are not being merged.');
8 8
9 if (window.testRunner) 9 if (window.testRunner)
10 testRunner.dumpAsText(); 10 testRunner.dumpAsText();
11 11
12 var rangeLeft = document.createRange(); 12 var rangeLeft = document.createRange();
13 var rangeRight = document.createRange(); 13 var rangeRight = document.createRange();
14 var elt = document.getElementById('elt').firstChild; 14 var elt = document.getElementById('elt').firstChild;
15 rangeLeft.setStart(elt, 0); 15 rangeLeft.setStart(elt, 0);
16 rangeLeft.setEnd(elt, 2); 16 rangeLeft.setEnd(elt, 2);
17 rangeRight.setStart(elt, 2); 17 rangeRight.setStart(elt, 2);
18 rangeRight.setEnd(elt, 4); 18 rangeRight.setEnd(elt, 4);
19 19
20 if (!window.internals) { 20 if (!window.internals) {
21 alert('This test requires window.interals to run!'); 21 alert('This test requires window.interals to run!');
22 } else { 22 } else {
23 window.internals.addTextMatchMarker(rangeLeft, true); 23 window.internals.addTextMatchMarker(rangeLeft, 'kActive');
24 window.internals.addTextMatchMarker(rangeRight, true); 24 window.internals.addTextMatchMarker(rangeRight, 'kActive');
25 25
26 shouldBe('window.internals.markerCountForNode(elt, "textmatch")', '2'); 26 shouldBe('window.internals.markerCountForNode(elt, "textmatch")', '2');
27 } 27 }
28 28
29 var successfullyParsed = true; 29 var successfullyParsed = true;
30 </script> 30 </script>
31 </body> 31 </body>
32 </html> 32 </html>
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/fast/dom/documentmarker-set-active.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698