CSSValue Immediates: Replace CSSPrimitiveValue usage with const references

Source/core/css/StylePropertySet.h

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2008, 2012 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 52 matching lines @@
         {
         }
 
         CSSPropertyID id() const { return static_cast<CSSPropertyID>(propertyMetadata().m_propertyID); }
         CSSPropertyID shorthandID() const { return propertyMetadata().shorthandID(); }
 
         bool isImportant() const { return propertyMetadata().m_important; }
         bool isInherited() const { return propertyMetadata().m_inherited; }
         bool isImplicit() const { return propertyMetadata().m_implicit; }
 
-        const CSSValue value() const { return propertyValue(); }
         // FIXME: We should try to remove this mutable overload.
-        CSSValue value() { return propertyValue(); }
+        const CSSValue& value() { return propertyValue(); }
 
         // FIXME: Remove this.
         CSSProperty toCSSProperty() const { return CSSProperty(propertyMetadata(), propertyValue()); }
 
         const StylePropertyMetadata& propertyMetadata() const;
 
     private:
-        const CSSValue propertyValue() const;
+        const CSSValue& propertyValue() const;
 
         const StylePropertySet& m_propertySet;
         unsigned m_index;
     };
 
     unsigned propertyCount() const;
     bool isEmpty() const;
     PropertyReference propertyAt(unsigned index) const { return PropertyReference(*this, index); }
     int findPropertyIndex(CSSPropertyID) const;
     bool hasProperty(CSSPropertyID property) const { return findPropertyIndex(property) != -1; }
 
-    NullableCSSValue getPropertyCSSValue(CSSPropertyID) const;
+    const NullableCSSValue getPropertyCSSValue(CSSPropertyID) const;
     String getPropertyValue(CSSPropertyID) const;
 
     bool propertyIsImportant(CSSPropertyID) const;
     CSSPropertyID getPropertyShorthand(CSSPropertyID) const;
     bool isPropertyImplicit(CSSPropertyID) const;
 
     CSSParserMode cssParserMode() const { return static_cast<CSSParserMode>(m_cssParserMode); }
 
     PassRefPtrWillBeRawPtr<MutableStylePropertySet> mutableCopy() const;
     PassRefPtrWillBeRawPtr<ImmutableStylePropertySet> immutableCopyIfNeeded() const;
@@ skipping 31 matching lines @@
         : m_cssParserMode(cssParserMode)
         , m_isMutable(false)
         , m_arraySize(std::min(immutableArraySize, unsigned(MaxArraySize)))
     { }
 
     unsigned m_cssParserMode : 3;
     mutable unsigned m_isMutable : 1;
     unsigned m_arraySize : 28;
 
     friend class PropertySetCSSStyleDeclaration;
+
 };
 
 class CORE_EXPORT ImmutableStylePropertySet : public StylePropertySet {
 public:
     ~ImmutableStylePropertySet();
     static PassRefPtrWillBeRawPtr<ImmutableStylePropertySet> create(const CSSProperty* properties, unsigned count, CSSParserMode);
 
     unsigned propertyCount() const { return m_arraySize; }
 
     const CSSValue* valueArray() const;
@@ skipping 92 matching lines @@
     return toMutableStylePropertySet(set.get());
 }
 
 inline const StylePropertyMetadata& StylePropertySet::PropertyReference::propertyMetadata() const
 {
     if (m_propertySet.isMutable())
         return toMutableStylePropertySet(m_propertySet).m_propertyVector.at(m_index).metadata();
     return toImmutableStylePropertySet(m_propertySet).metadataArray()[m_index];
 }
 
-inline const CSSValue StylePropertySet::PropertyReference::propertyValue() const
+inline const CSSValue& StylePropertySet::PropertyReference::propertyValue() const
 {
     if (m_propertySet.isMutable())
         return toMutableStylePropertySet(m_propertySet).m_propertyVector.at(m_index).value();
     return toImmutableStylePropertySet(m_propertySet).valueArray()[m_index];
 }
 
 inline unsigned StylePropertySet::propertyCount() const
 {
     if (m_isMutable)
         return toMutableStylePropertySet(this)->m_propertyVector.size();
@@ skipping 21 matching lines @@
 inline int StylePropertySet::findPropertyIndex(CSSPropertyID propertyID) const
 {
     if (m_isMutable)
         return toMutableStylePropertySet(this)->findPropertyIndex(propertyID);
     return toImmutableStylePropertySet(this)->findPropertyIndex(propertyID);
 }
 
 } // namespace blink
 
 #endif // StylePropertySet_h