Remove code to collect rules when matching.

sky/engine/core/css/ElementRuleCollector.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011. All rights reserved.
@@ skipping 43 matching lines @@
 
 ElementRuleCollector::~ElementRuleCollector()
 {
 }
 
 MatchResult& ElementRuleCollector::matchedResult()
 {
     return m_result;
 }
 
-PassRefPtr<StyleRuleList> ElementRuleCollector::matchedStyleRuleList()
-{
-    ASSERT(m_mode == SelectorChecker::CollectingStyleRules);
-    return m_styleRuleList.release();
-}
-
-PassRefPtr<CSSRuleList> ElementRuleCollector::matchedCSSRuleList()
-{
-    ASSERT(m_mode == SelectorChecker::CollectingCSSRules);
-    return m_cssRuleList.release();
-}
-
 inline void ElementRuleCollector::addMatchedRule(const RuleData* rule, CascadeScope cascadeScope, CascadeOrder cascadeOrder, unsigned styleSheetIndex, const CSSStyleSheet* parentStyleSheet)
 {
     if (!m_matchedRules)
         m_matchedRules = adoptPtr(new Vector<MatchedRule, 32>);
     m_matchedRules->append(MatchedRule(rule, cascadeScope, cascadeOrder, styleSheetIndex, parentStyleSheet));
 }
 
 void ElementRuleCollector::clearMatchedRules()
 {
     if (!m_matchedRules)
         return;
     m_matchedRules->clear();
 }
 
-inline StyleRuleList* ElementRuleCollector::ensureStyleRuleList()
-{
-    if (!m_styleRuleList)
-        m_styleRuleList = StyleRuleList::create();
-    return m_styleRuleList.get();
-}
-
-inline StaticCSSRuleList* ElementRuleCollector::ensureRuleList()
-{
-    if (!m_cssRuleList)
-        m_cssRuleList = StaticCSSRuleList::create();
-    return m_cssRuleList.get();
-}
-
 void ElementRuleCollector::addElementStyleProperties(const StylePropertySet* propertySet, bool isCacheable)
 {
     if (!propertySet)
         return;
     m_result.ranges.lastAuthorRule = m_result.matchedProperties.size();
     if (m_result.ranges.firstAuthorRule == -1)
         m_result.ranges.firstAuthorRule = m_result.ranges.lastAuthorRule;
     m_result.addMatchedProperties(propertySet);
     if (!isCacheable)
         m_result.isCacheable = false;
@@ skipping 39 matching lines @@
         collectMatchingRulesForList(matchRequest.ruleSet->idRules(element.idForStyleResolution()), contextFlags, cascadeScope, cascadeOrder, matchRequest, ruleRange);
     if (element.isStyledElement() && element.hasClass()) {
         for (size_t i = 0; i < element.classNames().size(); ++i)
             collectMatchingRulesForList(matchRequest.ruleSet->classRules(element.classNames()[i]), contextFlags, cascadeScope, cascadeOrder, matchRequest, ruleRange);
     }
 
     collectMatchingRulesForList(matchRequest.ruleSet->tagRules(element.localName()), contextFlags, cascadeScope, cascadeOrder, matchRequest, ruleRange);
     collectMatchingRulesForList(matchRequest.ruleSet->universalRules(), contextFlags, cascadeScope, cascadeOrder, matchRequest, ruleRange);
 }
 
-CSSRuleList* ElementRuleCollector::nestedRuleList(CSSRule* rule)
-{
-    switch (rule->type()) {
-    case CSSRule::MEDIA_RULE:
-        return toCSSMediaRule(rule)->cssRules();
-    case CSSRule::KEYFRAMES_RULE:
-        return toCSSKeyframesRule(rule)->cssRules();
-    case CSSRule::SUPPORTS_RULE:
-        return toCSSSupportsRule(rule)->cssRules();
-    default:
-        return 0;
-    }
-}
-
-template<class CSSRuleCollection>
-CSSRule* ElementRuleCollector::findStyleRule(CSSRuleCollection* cssRules, StyleRule* styleRule)
-{
-    if (!cssRules)
-        return 0;
-    CSSRule* result = 0;
-    for (unsigned i = 0; i < cssRules->length() && !result; ++i) {
-        CSSRule* cssRule = cssRules->item(i);
-        CSSRule::Type cssRuleType = cssRule->type();
-        if (cssRuleType == CSSRule::STYLE_RULE) {
-            CSSStyleRule* cssStyleRule = toCSSStyleRule(cssRule);
-            if (cssStyleRule->styleRule() == styleRule)
-                result = cssRule;
-        } else {
-            result = findStyleRule(nestedRuleList(cssRule), styleRule);
-        }
-    }
-    return result;
-}
-
-void ElementRuleCollector::appendCSSOMWrapperForRule(CSSStyleSheet* parentStyleSheet, StyleRule* rule)
-{
-    // |parentStyleSheet| is 0 if and only if the |rule| is coming from User Agent. In this case,
-    // it is safe to create CSSOM wrappers without parentStyleSheets as they will be used only
-    // by inspector which will not try to edit them.
-    RefPtr<CSSRule> cssRule = nullptr;
-    if (parentStyleSheet)
-        cssRule = findStyleRule(parentStyleSheet, rule);
-    else
-        cssRule = rule->createCSSOMWrapper();
-    ASSERT(!parentStyleSheet || cssRule);
-    ensureRuleList()->rules().append(cssRule);
-}
-
 void ElementRuleCollector::sortAndTransferMatchedRules()
 {
     if (!m_matchedRules || m_matchedRules->isEmpty())
         return;
 
     sortMatchedRules();
 
     Vector<MatchedRule, 32>& matchedRules = *m_matchedRules;
-    if (m_mode == SelectorChecker::CollectingStyleRules) {
-        for (unsigned i = 0; i < matchedRules.size(); ++i)
-            ensureStyleRuleList()->m_list.append(matchedRules[i].ruleData()->rule());
-        return;
-    }
-
-    if (m_mode == SelectorChecker::CollectingCSSRules) {
-        for (unsigned i = 0; i < matchedRules.size(); ++i)
-            appendCSSOMWrapperForRule(const_cast<CSSStyleSheet*>(matchedRules[i].parentStyleSheet()), matchedRules[i].ruleData()->rule());
-        return;
-    }
 
     // Now transfer the set of matched rules over to our list of declarations.
     for (unsigned i = 0; i < matchedRules.size(); i++) {
         // FIXME: Matching should not modify the style directly.
         const RuleData* ruleData = matchedRules[i].ruleData();
         if (m_style && ruleData->containsAttributeSelector())
             m_style->setUnique();
         m_result.addMatchedProperties(&ruleData->rule()->properties());
     }
 }
@@ skipping 51 matching lines @@
     // 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