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

Unified Diff: third_party/WebKit/LayoutTests/editing/surrounding-text/surrounding-text.html

Issue 2928073002: Reland: Made surrounding text work for last word in a document (Closed)
Patch Set: Do not create surrounding text for control elements, updated layout tests Created 3 years, 6 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/editing/surrounding-text/surrounding-text-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/editing/surrounding-text/surrounding-text.html
diff --git a/third_party/WebKit/LayoutTests/editing/surrounding-text/surrounding-text.html b/third_party/WebKit/LayoutTests/editing/surrounding-text/surrounding-text.html
index 31ac7833dac9b193430f0aa300d44c0f999ec514..1ebfebf6db9e3a38eab98a12faa9b314ecb90090 100644
--- a/third_party/WebKit/LayoutTests/editing/surrounding-text/surrounding-text.html
+++ b/third_party/WebKit/LayoutTests/editing/surrounding-text/surrounding-text.html
@@ -54,8 +54,8 @@ function run() {
shouldBeEqualToString('surroundingText(\'<fieldset>.</fieldset>12345<button>abc</button><p>6789<br id="here"/>12345</p>6789<textarea>abc</textarea>0123<fieldset>.</fieldset>\', 0, 100)', '6789 12345 6789');
shouldBeEqualToString('surroundingText(\'<button>.</button><div id="here">This is <!-- comment --!>a test <\' + \'script language="javascript"><\' + \'/script>example<button>.</button>\', 0, 100)', 'This is a test example');
shouldBeEqualToString('surroundingText(\'<button>.</button><div id="here">012345678901234567890123456789</div><button>.</button>\', 15, 12)', '901234567890');
- shouldBeEqualToString('surroundingText(\'<option>.</option>12345<button id="here">test</button><option>.</option>\', 0, 100)', '');
- shouldBeEqualToString('surroundingText(\'<option>.</option>12345<button>te<span id="here">st</span></button><option>.</option>\', 0, 100)', '');
+ shouldBeEqualToString('surroundingText(\'<option>.</option>12345<button id="here">test</button><option>.</option>\', 1, 100)', '');
+ shouldBeEqualToString('surroundingText(\'<option>.</option>12345<button>te<span id="here">st</span></button><option>.</option>\', 1, 100)', '');
shouldBeEqualToString('surroundingText(\'<p id="here">.</p>\', 0, 2)', '.');
document.body.removeChild(document.getElementById('test'));
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/editing/surrounding-text/surrounding-text-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698