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

Unified Diff: third_party/WebKit/LayoutTests/css-parser/content-parsing-invalid.html

Issue 1837413002: Ensure that we don't allow 'none' alongside other values in content property (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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/css-parser/content-parsing-invalid.html
diff --git a/third_party/WebKit/LayoutTests/css-parser/content-parsing-invalid.html b/third_party/WebKit/LayoutTests/css-parser/content-parsing-invalid.html
index 6aa63571ce7b906fef3884dacd987c389ef7c4cb..a599d4f361e48266ba0e8c6560281669925e137d 100644
--- a/third_party/WebKit/LayoutTests/css-parser/content-parsing-invalid.html
+++ b/third_party/WebKit/LayoutTests/css-parser/content-parsing-invalid.html
@@ -3,19 +3,29 @@
<script src="../resources/testharnessreport.js"></script>
<style>
-#target::before {
+#target1::before {
content: normal normal;
}
-#target::after {
+#target1::after {
content: normal normal;
}
+
+#target2::before {
+ content: none open-quote;
+}
+#target2::after {
+ content: url("a") none;
+}
</style>
-<div id="target"></div>
+<div id="target1"></div>
+<div id="target2"></div>
<script>
test(function(){
- assert_equals(getComputedStyle(target, '::before').content, '');
- assert_equals(getComputedStyle(target, '::after').content, '');
+ assert_equals(document.styleSheets[0].cssRules[0].style.content, '');
+ assert_equals(document.styleSheets[0].cssRules[1].style.content, '');
+ assert_equals(document.styleSheets[0].cssRules[2].style.content, '');
+ assert_equals(document.styleSheets[0].cssRules[3].style.content, '');
}, "Test to ensure that invalid values of content on pseudo elements are dropped by the parser");
</script>

Powered by Google App Engine
This is Rietveld 408576698