Sibling invalidation sets

Source/core/css/invalidation/StyleSheetInvalidationAnalysis.cpp

Index: Source/core/css/invalidation/StyleSheetInvalidationAnalysis.cpp
diff --git a/Source/core/css/invalidation/StyleSheetInvalidationAnalysis.cpp b/Source/core/css/invalidation/StyleSheetInvalidationAnalysis.cpp
index 5a1a12953e44de843c89abf2e797aa54b65d105b..3543eb58b48aa0fb2584fd52a5e7aecc4e4dfeef 100644
--- a/Source/core/css/invalidation/StyleSheetInvalidationAnalysis.cpp
+++ b/Source/core/css/invalidation/StyleSheetInvalidationAnalysis.cpp
@@ -61,8 +61,6 @@ static bool determineSelectorScopes(const CSSSelectorList& selectorList, HashSet
             // scopeSelector. Currently requests full style recalc.
             if (relation == CSSSelector::ShadowDeep || relation == CSSSelector::ShadowPseudo)
                 return false;
-            if (relation != CSSSelector::Descendant && relation != CSSSelector::Child && relation != CSSSelector::SubSelector)

[Timothy Loh, 2015/09/10 06:09:40]

This doesn't seem related?

[rune, 2015/09/10 13:54:50]

This might actually work with the current code given that
maxDirectAdjacentSelectors is up-to-date with the new stylesheet when the
SubtreeStyleChange set in StyleSheetInvalidationAnalysis::invalidateStyle
triggers sibling recalcs in
ContainerNode::checkForChildrenAdjacentRuleChanges().

But, it should be a different CL in either case.

[Eric Willigers, 2015/09/14 07:20:24]

Reverted, it isn't needed.

-                break;
         }
         if (!scopeSelector)
             return false;