LayoutTests/editing/spelling/spelling-insert-html.html - Issue 43803002: dumpAsText() is unnecessarily called in spellchecking tests

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: LayoutTests/editing/spelling/spelling-insert-html.html

Issue 43803002: dumpAsText() is unnecessarily called in spellchecking tests (Closed) Base URL: https://chromium.googlesource.com/chromium/blink@master

Patch Set: Created 7 years, 2 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

OLD	NEW
1 <!DOCTYPE html>	1 <!DOCTYPE html>

2 <html>	2 <html>

3 <head>	3 <head>

4 <script src="../../fast/js/resources/js-test-pre.js"></script>	4 <script src="../../fast/js/resources/js-test-pre.js"></script>

5 </head>	5 </head>

6 <body>	6 <body>

7 <pre id="description"></pre>	7 <pre id="description"></pre>

8 <pre id="console"></pre>	8 <pre id="console"></pre>

9 <div id="container">	9 <div id="container">

10 <div id="destination" contentEditable>food.</div>	10 <div id="destination" contentEditable>food.</div>

(...skipping 18 matching lines...) Expand all Loading...
29 var marked = internals.markerRangeForNode(texts[i], "spelling", 0);	29 var marked = internals.markerRangeForNode(texts[i], "spelling", 0);

30 if (marked)	30 if (marked)

31 markedText += marked.toString()	31 markedText += marked.toString()

32 }	32 }

33	33

34 // The first "foo" isn't checked because it crosses the pasted and base html .	34 // The first "foo" isn't checked because it crosses the pasted and base html .

35 // See http://webkit.org/b/66450.	35 // See http://webkit.org/b/66450.

36 shouldBeEqualToString("markedText", "zz");	36 shouldBeEqualToString("markedText", "zz");

37 }	37 }

38	38

39 if (window.testRunner)

40 testRunner.dumpAsText();

41

42 </script>	39 </script>

43 <script src="../../fast/js/resources/js-test-post.js"></script>	40 <script src="../../fast/js/resources/js-test-post.js"></script>

44 </body>	41 </body>

45 </html>	42 </html>

OLD	NEW