commit 8affd37880f91d33a345513b25b85e71ec011fdf
author abarth@chromium.org <abarth@chromium.org@bbb929c8-8fbe-4397-9dbb-9b2b20218538> Thu Jun 20 21:12:25 2013
committer abarth@chromium.org <abarth@chromium.org@bbb929c8-8fbe-4397-9dbb-9b2b20218538> Thu Jun 20 21:12:25 2013
tree a5108e9288dc26a00e3528317a508ae7b5629973
parent d1dc33d638f4657d4b4563de30d8f2bc089f8850

RuleSet::AtomRuleMap should be private

This CL makes some functions on RuleSet private in order to hide the definition
of AtomRuleMap. These functions aren't used outside of this class.

R=ojan

Review URL: https://chromiumcodereview.appspot.com/17351012

git-svn-id: svn://svn.chromium.org/blink/trunk@152824 bbb929c8-8fbe-4397-9dbb-9b2b20218538

Source/core/css/RuleSet.cpp

diff --git a/Source/core/css/RuleSet.cpp b/Source/core/css/RuleSet.cpp
index 48ed830..a08c06c 100644
--- a/Source/core/css/RuleSet.cpp
+++ b/Source/core/css/RuleSet.cpp
@@ -379,7 +379,7 @@
         addRule(rule, selectorIndex, addRuleFlags);
 }
 
-static inline void shrinkMapVectorsToFit(RuleSet::AtomRuleMap& map)
+void RuleSet::shrinkMapVectorsToFit(AtomRuleMap& map)
 {
     RuleSet::AtomRuleMap::iterator end = map.end();
     for (RuleSet::AtomRuleMap::iterator it = map.begin(); it != end; ++it)

Source/core/css/RuleSet.h

diff --git a/Source/core/css/RuleSet.h b/Source/core/css/RuleSet.h
index 0b01802..0d9e999 100644
--- a/Source/core/css/RuleSet.h
+++ b/Source/core/css/RuleSet.h
@@ -106,15 +106,9 @@
 public:
     static PassOwnPtr<RuleSet> create() { return adoptPtr(new RuleSet); }
 
-    typedef HashMap<AtomicStringImpl*, OwnPtr<Vector<RuleData> > > AtomRuleMap;
-
     void addRulesFromSheet(StyleSheetContents*, const MediaQueryEvaluator&, StyleResolver* = 0, const ContainerNode* = 0);
-
     void addStyleRule(StyleRule*, AddRuleFlags);
     void addRule(StyleRule*, unsigned selectorIndex, AddRuleFlags);
-    void addPageRule(StyleRulePage*);
-    void addToRuleSet(AtomicStringImpl* key, AtomRuleMap&, const RuleData&);
-    void addRegionRule(StyleRuleRegion*, bool hasDocumentSecurityOrigin);
 
     const RuleFeatureSet& features() const { return m_features; }
 
@@ -151,16 +145,23 @@
     Vector<RuleSetSelectorPair> m_regionSelectorsAndRuleSets;
 
 private:
+    typedef HashMap<AtomicStringImpl*, OwnPtr<Vector<RuleData> > > AtomRuleMap;
+
     RuleSet()
         : m_ruleCount(0)
         , m_hasDirtyRules(false)
     {
     }
 
+    void addToRuleSet(AtomicStringImpl* key, AtomRuleMap&, const RuleData&);
+    void addPageRule(StyleRulePage*);
+    void addRegionRule(StyleRuleRegion*, bool hasDocumentSecurityOrigin);
+
     void addChildRules(const Vector<RefPtr<StyleRuleBase> >&, const MediaQueryEvaluator& medium, StyleResolver*, const ContainerNode* scope, bool hasDocumentSecurityOrigin, AddRuleFlags);
     bool findBestRuleSetAndAdd(const CSSSelector*, RuleData&);
 
     void shrinkToFit();
+    static inline void shrinkMapVectorsToFit(AtomRuleMap&);
 
     AtomRuleMap m_idRules;
     AtomRuleMap m_classRules;