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

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/editing/run/inserttext-expected.txt

Issue 2718543003: Remove EditingAppleTabSpan class handling (Closed)
Patch Set: Make editing/inserting/5549929-2.html pass Created 3 years, 9 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/external/wpt/editing/run/inserttext-expected.txt
diff --git a/third_party/WebKit/LayoutTests/external/wpt/editing/run/inserttext-expected.txt b/third_party/WebKit/LayoutTests/external/wpt/editing/run/inserttext-expected.txt
index 107b148f2cbbdff114898af085552a5f2adc0b18..1d4e1d92c05d91d66b46fb655d9899d0307bc0ce 100644
--- a/third_party/WebKit/LayoutTests/external/wpt/editing/run/inserttext-expected.txt
+++ b/third_party/WebKit/LayoutTests/external/wpt/editing/run/inserttext-expected.txt
@@ -20,7 +20,7 @@ PASS [["inserttext",""]] "foo[bar]baz" queryCommandState("inserttext") after
PASS [["inserttext",""]] "foo[bar]baz" queryCommandValue("inserttext") after
PASS [["inserttext","\t"]] "foo[]bar": execCommand("inserttext", false, "\t") return value
PASS [["inserttext","\t"]] "foo[]bar" checks for modifications to non-editable content
-FAIL [["inserttext","\t"]] "foo[]bar" compare innerHTML assert_equals: Unexpected innerHTML (after normalizing inline style) expected "foo\tbar" but got "foo<span class=\"Apple-tab-span\" style=\"white-space:pre\">\t</span>bar"
+FAIL [["inserttext","\t"]] "foo[]bar" compare innerHTML assert_equals: Unexpected innerHTML (after normalizing inline style) expected "foo\tbar" but got "foo<span style=\"white-space:pre\">\t</span>bar"
PASS [["inserttext","\t"]] "foo[]bar" queryCommandIndeterm("inserttext") before
PASS [["inserttext","\t"]] "foo[]bar" queryCommandState("inserttext") before
PASS [["inserttext","\t"]] "foo[]bar" queryCommandValue("inserttext") before
@@ -1038,7 +1038,7 @@ PASS [["inserttext","a"]] "http://a[]" queryCommandState("inserttext") after
PASS [["inserttext","a"]] "http://a[]" queryCommandValue("inserttext") after
PASS [["inserttext","\t"]] "http://a[]": execCommand("inserttext", false, "\t") return value
PASS [["inserttext","\t"]] "http://a[]" checks for modifications to non-editable content
-FAIL [["inserttext","\t"]] "http://a[]" compare innerHTML assert_equals: Unexpected innerHTML (after normalizing inline style) expected "<a href=\"http://a\">http://a</a>\t" but got "http://a<span class=\"Apple-tab-span\" style=\"white-space:pre\">\t</span>"
+FAIL [["inserttext","\t"]] "http://a[]" compare innerHTML assert_equals: Unexpected innerHTML (after normalizing inline style) expected "<a href=\"http://a\">http://a</a>\t" but got "http://a<span style=\"white-space:pre\">\t</span>"
yosin_UTC9 2017/03/07 05:53:17 It seems we don't need to add |style="white-space:
joone 2017/03/07 19:12:59 Yes, I think so. Firefox also doesn't add a <span>
joone 2017/03/11 00:51:43 yosin@ do we need to create <a> for "http://a" whi
PASS [["inserttext","\t"]] "http://a[]" queryCommandIndeterm("inserttext") before
PASS [["inserttext","\t"]] "http://a[]" queryCommandState("inserttext") before
PASS [["inserttext","\t"]] "http://a[]" queryCommandValue("inserttext") before

Powered by Google App Engine
This is Rietveld 408576698