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

Unified Diff: sky/engine/core/css/ElementRuleCollector.cpp

Issue 796583002: Use the HashSet of attribute names in SharedStyleFinder. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Use C++11 loops. Created 6 years 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 | « sky/engine/core/css/ElementRuleCollector.h ('k') | sky/engine/core/css/RuleFeature.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/core/css/ElementRuleCollector.cpp
diff --git a/sky/engine/core/css/ElementRuleCollector.cpp b/sky/engine/core/css/ElementRuleCollector.cpp
index f4ad0a25354d51f874d5b8fb27e003a24ea6b2aa..d9f5f38b0f1cbdb015d004e5fdebf53fc78fa02d 100644
--- a/sky/engine/core/css/ElementRuleCollector.cpp
+++ b/sky/engine/core/css/ElementRuleCollector.cpp
@@ -191,20 +191,4 @@ void ElementRuleCollector::sortMatchedRules()
std::sort(m_matchedRules->begin(), m_matchedRules->end(), compareRules);
}
-bool ElementRuleCollector::hasAnyMatchingRules(RuleSet* ruleSet)
-{
- clearMatchedRules();
-
- m_mode = SelectorChecker::SharingRules;
- // To check whether a given RuleSet has any rule matching a given element,
- // should not see the element's treescope. Because RuleSet has no
- // information about "scope".
- int firstRuleIndex = -1, lastRuleIndex = -1;
- RuleRange ruleRange(firstRuleIndex, lastRuleIndex);
- // FIXME: Verify whether it's ok to ignore CascadeScope here.
- collectMatchingRules(MatchRequest(ruleSet), ruleRange, SelectorChecker::DefaultBehavior);
-
- return m_matchedRules && !m_matchedRules->isEmpty();
-}
-
} // namespace blink
« no previous file with comments | « sky/engine/core/css/ElementRuleCollector.h ('k') | sky/engine/core/css/RuleFeature.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698