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

Unified Diff: third_party/WebKit/LayoutTests/editing/execCommand/script-tests/toggle-style-2.js

Issue 2800723002: Ensure we never remove the style attribute when syncing it from CSSOM. (Closed)
Patch Set: Nits 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/editing/execCommand/script-tests/toggle-style-2.js
diff --git a/third_party/WebKit/LayoutTests/editing/execCommand/script-tests/toggle-style-2.js b/third_party/WebKit/LayoutTests/editing/execCommand/script-tests/toggle-style-2.js
index e83827a0e8d0e610f9d7733442bce83ae5866980..d236646e4936683845a6fa3382b46ce351851558 100644
--- a/third_party/WebKit/LayoutTests/editing/execCommand/script-tests/toggle-style-2.js
+++ b/third_party/WebKit/LayoutTests/editing/execCommand/script-tests/toggle-style-2.js
@@ -30,7 +30,7 @@ function testDoubleToggle(toggleCommand, initialContents, expectedContents)
}
testSingleToggle("underline", "test", "<u>test</u>");
-testSingleToggle("underline", "<u><b><strike>test</strike></b></u>", "<b><strike>test</strike></b>");
+testSingleToggle("underline", "<u><b><strike>test</strike></b></u>", "<b style=\"\"><strike style=\"\">test</strike></b>");
testDoubleToggle("underline", "test", "test");
testSingleToggle("strikethrough", "test", "<strike>test</strike>");
testSingleToggle("strikethrough", "<u><b><strike>test</strike></b></u>", "<u><b>test</b></u>");

Powered by Google App Engine
This is Rietveld 408576698