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

Unified Diff: Source/core/dom/shadow/SelectRuleFeatureSet.h

Issue 598093005: Simplify SelectRuleFeatureSet class. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Rebased Created 6 years, 3 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 | Source/core/dom/shadow/SelectRuleFeatureSet.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/dom/shadow/SelectRuleFeatureSet.h
diff --git a/Source/core/dom/shadow/SelectRuleFeatureSet.h b/Source/core/dom/shadow/SelectRuleFeatureSet.h
index 0593133e52532c2240c792bb8a11129aa4093710..04fd92f1ccf5b7338d882993249f5b408b7becd3 100644
--- a/Source/core/dom/shadow/SelectRuleFeatureSet.h
+++ b/Source/core/dom/shadow/SelectRuleFeatureSet.h
@@ -38,58 +38,13 @@ namespace blink {
class SpaceSplitString;
-class SelectRuleFeatureSet {
+class SelectRuleFeatureSet : public RuleFeatureSet {
DISALLOW_ALLOCATION();
public:
- void add(const SelectRuleFeatureSet&);
- void clear();
- void collectFeaturesFromSelector(const CSSSelector&);
-
- bool hasSelectorForId(const AtomicString&) const;
- bool hasSelectorForClass(const AtomicString&) const;
- bool hasSelectorForAttribute(const AtomicString&) const;
- bool hasSelectorForPseudoType(CSSSelector::PseudoType) const;
-
- bool hasSelectorForChecked() const { return hasSelectorForPseudoType(CSSSelector::PseudoChecked); }
- bool hasSelectorForEnabled() const { return hasSelectorForPseudoType(CSSSelector::PseudoEnabled); }
- bool hasSelectorForDisabled() const { return hasSelectorForPseudoType(CSSSelector::PseudoDisabled); }
- bool hasSelectorForIndeterminate() const { return hasSelectorForPseudoType(CSSSelector::PseudoIndeterminate); }
- bool hasSelectorForLink() const { return hasSelectorForPseudoType(CSSSelector::PseudoLink); }
- bool hasSelectorForTarget() const { return hasSelectorForPseudoType(CSSSelector::PseudoTarget); }
- bool hasSelectorForVisited() const { return hasSelectorForPseudoType(CSSSelector::PseudoVisited); }
-
bool checkSelectorsForClassChange(const SpaceSplitString& changedClasses) const;
bool checkSelectorsForClassChange(const SpaceSplitString& oldClasses, const SpaceSplitString& newClasses) const;
-
- void trace(Visitor* visitor) { visitor->trace(m_cssRuleFeatureSet); }
-
-private:
- RuleFeatureSet m_cssRuleFeatureSet;
};
-inline bool SelectRuleFeatureSet::hasSelectorForId(const AtomicString& idValue) const
-{
- ASSERT(!idValue.isEmpty());
- return m_cssRuleFeatureSet.hasSelectorForId(idValue);
-}
-
-inline bool SelectRuleFeatureSet::hasSelectorForClass(const AtomicString& classValue) const
-{
- ASSERT(!classValue.isEmpty());
- return m_cssRuleFeatureSet.hasSelectorForClass(classValue);
-}
-
-inline bool SelectRuleFeatureSet::hasSelectorForAttribute(const AtomicString& attributeName) const
-{
- ASSERT(!attributeName.isEmpty());
- return m_cssRuleFeatureSet.hasSelectorForAttribute(attributeName);
-}
-
-inline bool SelectRuleFeatureSet::hasSelectorForPseudoType(CSSSelector::PseudoType pseudo) const
-{
- return m_cssRuleFeatureSet.hasSelectorForPseudoType(pseudo);
-}
-
}
#endif
« no previous file with comments | « no previous file | Source/core/dom/shadow/SelectRuleFeatureSet.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698