Have CSSFontFeatureValue store the tag as an AtomicString instead of a String

Source/core/css/CSSFontFeatureValue.h

Index: Source/core/css/CSSFontFeatureValue.h
diff --git a/Source/core/css/CSSFontFeatureValue.h b/Source/core/css/CSSFontFeatureValue.h
index 011cd1e438539676b3ec87127fe84b830e149b39..3fa35e7401558f21995708b3c816c58f90a56c1b 100644
--- a/Source/core/css/CSSFontFeatureValue.h
+++ b/Source/core/css/CSSFontFeatureValue.h
@@ -33,21 +33,21 @@ namespace WebCore {
 
 class CSSFontFeatureValue : public CSSValue {
 public:
-    static PassRefPtr<CSSFontFeatureValue> create(const String& tag, int value)
+    static PassRefPtr<CSSFontFeatureValue> create(const AtomicString& tag, int value)

[eseidel, 2013/11/27 20:37:34]

I'm not sure what "value" is, but that's really a separate topic for another
day.

     {
         return adoptRef(new CSSFontFeatureValue(tag, value));
     }
 
-    const String& tag() const { return m_tag; }
+    const AtomicString& tag() const { return m_tag; }
     int value() const { return m_value; }
     String customCSSText() const;
 
     bool equals(const CSSFontFeatureValue&) const;
 
 private:
-    CSSFontFeatureValue(const String&, int);
+    CSSFontFeatureValue(const AtomicString&, int);

[eseidel, 2013/11/27 20:37:34]

I'd probably add back the "tag" and "value" labels since this constructor is
very unclear as to what it takes.

[Inactive, 2013/11/27 20:44:49]

Done.

 
-    String m_tag;
+    AtomicString m_tag;
     const int m_value;
 };