 Chromium Code Reviews
 Chromium Code Reviews Issue 129633003:
  Add a first pass of a class descendant invalidator, and a containing RuleSetAnalyzer  (Closed) 
  Base URL: svn://svn.chromium.org/blink/trunk
    
  
    Issue 129633003:
  Add a first pass of a class descendant invalidator, and a containing RuleSetAnalyzer  (Closed) 
  Base URL: svn://svn.chromium.org/blink/trunk| Index: Source/core/css/RuleFeature.h | 
| diff --git a/Source/core/css/RuleFeature.h b/Source/core/css/RuleFeature.h | 
| index b51863c428975a72a41f1295c091d558f12976a3..211ee143f16f6cc26cb51f539054c025c47708d1 100644 | 
| --- a/Source/core/css/RuleFeature.h | 
| +++ b/Source/core/css/RuleFeature.h | 
| @@ -22,6 +22,7 @@ | 
| #ifndef RuleFeature_h | 
| #define RuleFeature_h | 
| +#include "core/css/analyzer/RuleSetAnalyzer.h" | 
| #include "wtf/Forward.h" | 
| #include "wtf/HashSet.h" | 
| #include "wtf/text/AtomicStringHash.h" | 
| @@ -80,12 +81,18 @@ public: | 
| return idsInRules.contains(idValue); | 
| } | 
| + const RuleSetAnalyzer* getRuleSetAnalyzer() const; | 
| 
esprehn
2014/01/15 04:27:32
Still named wrong.
 
chrishtr
2014/01/15 18:29:16
Done.
 | 
| + RuleSetAnalyzer* ensureRuleSetAnalyzer(); | 
| + | 
| + // FIXME: move this stuff into the rule set analyzer. | 
| HashSet<AtomicString> idsInRules; | 
| HashSet<AtomicString> classesInRules; | 
| HashSet<AtomicString> attrsInRules; | 
| Vector<RuleFeature> siblingRules; | 
| Vector<RuleFeature> uncommonAttributeRules; | 
| private: | 
| + RefPtr<RuleSetAnalyzer> ruleSetAnalyzer; | 
| 
esprehn
2014/01/15 04:27:32
Missing m_ prefix.
 
chrishtr
2014/01/15 18:28:21
Done.
 | 
| + | 
| void collectFeaturesFromSelectorList(const CSSSelectorList*); | 
| bool m_usesFirstLineRules; |