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

Unified Diff: third_party/WebKit/Source/core/css/parser/CSSSelectorParserTest.cpp

Issue 2838013005: Update CSSSelectorParserTest so that it doesn't test /deep/ nor ::shadow (Closed)
Patch Set: 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/css/parser/CSSSelectorParserTest.cpp
diff --git a/third_party/WebKit/Source/core/css/parser/CSSSelectorParserTest.cpp b/third_party/WebKit/Source/core/css/parser/CSSSelectorParserTest.cpp
index 4cad4aef7a90c503bdac0d68a51f5836710cd481..ad9b91e82bcd511f6d5013654dfe0fdc5aed3da8 100644
--- a/third_party/WebKit/Source/core/css/parser/CSSSelectorParserTest.cpp
+++ b/third_party/WebKit/Source/core/css/parser/CSSSelectorParserTest.cpp
@@ -105,9 +105,7 @@ TEST(CSSSelectorParserTest, InvalidANPlusB) {
}
TEST(CSSSelectorParserTest, ShadowDomPseudoInCompound) {
- const char* test_cases[][2] = {{"::shadow", "::shadow"},
- {".a::shadow", ".a::shadow"},
- {"::content", "::content"},
+ const char* test_cases[][2] = {{"::content", "::content"},
{".a::content", ".a::content"},
{"::content.a", "::content.a"},
{"::content.a.b", "::content.a.b"},
@@ -127,16 +125,12 @@ TEST(CSSSelectorParserTest, ShadowDomPseudoInCompound) {
TEST(CSSSelectorParserTest, PseudoElementsInCompoundLists) {
const char* test_cases[] = {":not(::before)",
":not(::content)",
- ":not(::shadow)",
":host(::before)",
":host(::content)",
- ":host(::shadow)",
":host-context(::before)",
":host-context(::content)",
- ":host-context(::shadow)",
":-webkit-any(::after, ::before)",
- ":-webkit-any(::content, span)",
- ":-webkit-any(div, ::shadow)"};
+ ":-webkit-any(::content, span)"};
for (auto test_case : test_cases) {
CSSTokenizer tokenizer(test_case);
@@ -201,9 +195,7 @@ TEST(CSSSelectorParserTest, WorkaroundForInvalidCustomPseudoInUAStyle) {
}
TEST(CSSSelectorParserTest, ValidPseudoElementInNonRightmostCompound) {
- const char* test_cases[] = {"::content *", "::shadow *",
- "::content div::before",
- "::shadow ::first-letter"};
+ const char* test_cases[] = {"::content *", "::content div::before"};
for (auto test_case : test_cases) {
CSSTokenizer tokenizer(test_case);
@@ -247,18 +239,14 @@ TEST(CSSSelectorParserTest, SerializedUniversal) {
const char* test_cases[][2] = {
{"*::-webkit-volume-slider", "::-webkit-volume-slider"},
{"*::cue(i)", "::cue(i)"},
- {"*::shadow", "::shadow"},
{"*:host-context(.x)", "*:host-context(.x)"},
{"*:host", "*:host"},
{"|*::-webkit-volume-slider", "|*::-webkit-volume-slider"},
{"|*::cue(i)", "|*::cue(i)"},
- {"|*::shadow", "|*::shadow"},
{"*|*::-webkit-volume-slider", "::-webkit-volume-slider"},
{"*|*::cue(i)", "::cue(i)"},
- {"*|*::shadow", "::shadow"},
{"ns|*::-webkit-volume-slider", "ns|*::-webkit-volume-slider"},
- {"ns|*::cue(i)", "ns|*::cue(i)"},
- {"ns|*::shadow", "ns|*::shadow"}};
+ {"ns|*::cue(i)", "ns|*::cue(i)"}};
CSSParserContext* context = CSSParserContext::Create(kHTMLStandardMode);
StyleSheetContents* sheet = StyleSheetContents::Create(context);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698