Avoid reparsing keyframe selectors

Source/core/css/CSSKeyframeRule.h

Index: Source/core/css/CSSKeyframeRule.h
diff --git a/Source/core/css/CSSKeyframeRule.h b/Source/core/css/CSSKeyframeRule.h
index 365ef74c1bb730d263f758cbca87805c6e037d3b..754b8fe7b2cd2db7ee014d3e12b755f5006eeaec 100644
--- a/Source/core/css/CSSKeyframeRule.h
+++ b/Source/core/css/CSSKeyframeRule.h
@@ -45,10 +45,11 @@ public:
     }
     ~StyleKeyframe();
 
-    String keyText() const { return m_key; }
-    void setKeyText(const String& s) { m_key = s; }
+    String keyText() const;
+    void setKeyText(const String&);
+    void setKeys(const Vector<float>&);
 
-    void getKeys(Vector<float>& keys) const   { parseKeyString(m_key, keys); }
+    const Vector<float>& getKeys() const { return m_keys; }

[apavlov, 2013/07/04 22:02:52]

In WebKit/Blink, getFoo(...) usually fills an output parameter. Normally,
getters do not have the "get" prefix. You can fix this in a subsequent patch.

 
     const StylePropertySet* properties() const { return m_properties.get(); }
     MutableStylePropertySet* mutableProperties();
@@ -61,12 +62,11 @@ public:
 private:
     StyleKeyframe();
 
-    static void parseKeyString(const String&, Vector<float>& keys);
+    static void parseKeyString(const String&, Vector<float>&);
 
     RefPtr<StylePropertySet> m_properties;
-    // FIXME: This should be a parsed vector of floats.
-    // comma separated list of keys
-    String m_key;
+    mutable String m_keyText;
+    Vector<float> m_keys;
 };
 
 class CSSKeyframeRule : public CSSRule {