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

Unified Diff: third_party/WebKit/LayoutTests/editing/undo/remove-css-property-and-remove-style-expected.txt

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/undo/remove-css-property-and-remove-style-expected.txt
diff --git a/third_party/WebKit/LayoutTests/editing/undo/remove-css-property-and-remove-style-expected.txt b/third_party/WebKit/LayoutTests/editing/undo/remove-css-property-and-remove-style-expected.txt
index b6173f1c4a9f90fbdb5f72d56361e7fc8ab3d954..525f4e90d641746141cbd7367fbb3d5844b20e24 100644
--- a/third_party/WebKit/LayoutTests/editing/undo/remove-css-property-and-remove-style-expected.txt
+++ b/third_party/WebKit/LayoutTests/editing/undo/remove-css-property-and-remove-style-expected.txt
@@ -1,4 +1,4 @@
-This tests removing style attribute after removing CSS property. Undo should bring back the CSS property we removed.
+This tests removing style after removing CSS property. Undo should bring back the CSS property we removed.
Initially "test" should be bold wrapped with a span:
| <span>
@@ -8,7 +8,7 @@ Initially "test" should be bold wrapped with a span:
Unbolding should remove the CSS style and also remove the span:
| "<#selection-anchor>test<#selection-focus>"
-Undo should bring back both the span and style attribute so that "test" is once again bold:
+Undo should bring back both the span and style so that "test" is once again bold:
| <span>
| style="font-weight: 900;"
| "<#selection-anchor>test<#selection-focus>"
@@ -31,19 +31,21 @@ Reset, and added id and title:
Unbolding should remove the CSS style but shouldn't remove the span:
| <span>
| id="test_span"
+| style=""
| title="hello, world"
| "<#selection-anchor>test<#selection-focus>"
-Undo should restore the style attribute and "test" should be bold:
+Undo should restore the style and "test" should be bold:
| <span>
| id="test_span"
| style="font-weight: 900;"
| title="hello, world"
| "<#selection-anchor>test<#selection-focus>"
-Redo should remove the style attribute again:
+Redo should remove the style again:
| <span>
| id="test_span"
+| style=""
| title="hello, world"
| "<#selection-anchor>test<#selection-focus>"
@@ -52,12 +54,12 @@ Reset, and added color:blue:
| style="font-weight: 900; color: blue;"
| "test"
-Unbolding should remove the font-weight but shouldn't remove the style attribute:
+Unbolding should remove the font-weight but shouldn't remove the style :
| <span>
| style="color: blue;"
| "<#selection-anchor>test<#selection-focus>"
-Undo should reset the style attribute so that "test" is both bold and blue:
+Undo should reset the style so that "test" is both bold and blue:
| <span>
| style="color: blue; font-weight: 900;"
| "<#selection-anchor>test<#selection-focus>"
@@ -67,7 +69,7 @@ Redo should only remove font-weight and leave "test" blue:
| style="color: blue;"
| "<#selection-anchor>test<#selection-focus>"
-Setting the forecolor to black should remove both the style attribute and the span:
+Setting the forecolor to black should remove both the style and the span:
| "<#selection-anchor>test<#selection-focus>"
Undo should make "test" blue again:

Powered by Google App Engine
This is Rietveld 408576698