CSSValue Immediates: Change CSSValue to an object instead of a pointer

Source/core/css/CSSProperty.h

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2005, 2006 Apple Computer, Inc.
  *
  * 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 38 matching lines @@
     uint16_t m_isSetFromShorthand : 1;
     uint16_t m_indexInShorthandsVector : 2; // If this property was set as part of an ambiguous shorthand, gives the index in the shorthands vector.
     uint16_t m_important : 1;
     uint16_t m_implicit : 1; // Whether or not the property was set implicitly as the result of a shorthand.
     uint16_t m_inherited : 1;
 };
 
 class CSSProperty {
     ALLOW_ONLY_INLINE_ALLOCATION();
 public:
-    CSSProperty(CSSPropertyID propertyID, PassRefPtrWillBeRawPtr<CSSValue> value, bool important = false, bool isSetFromShorthand = false, int indexInShorthandsVector = 0, bool implicit = false)
+    CSSProperty(CSSPropertyID propertyID, CSSValue value, bool important = false, bool isSetFromShorthand = false, int indexInShorthandsVector = 0, bool implicit = false)
         : m_metadata(propertyID, isSetFromShorthand, indexInShorthandsVector, important, implicit, CSSPropertyMetadata::isInheritedProperty(propertyID))
         , m_value(value)
     {
     }
 
     // FIXME: Remove this.
-    CSSProperty(StylePropertyMetadata metadata, CSSValue* value)
+    CSSProperty(StylePropertyMetadata metadata, CSSValue value)
         : m_metadata(metadata)
         , m_value(value)
     {
     }
 
     CSSPropertyID id() const { return static_cast<CSSPropertyID>(m_metadata.m_propertyID); }
     bool isSetFromShorthand() const { return m_metadata.m_isSetFromShorthand; }
     CSSPropertyID shorthandID() const { return m_metadata.shorthandID(); }
     bool isImportant() const { return m_metadata.m_important; }
 
-    CSSValue* value() const { return m_value.get(); }
+    const CSSValue& value() const { return m_value; }
 
     static CSSPropertyID resolveDirectionAwareProperty(CSSPropertyID, TextDirection, WritingMode);
     static bool isAffectedByAllProperty(CSSPropertyID);
 
     const StylePropertyMetadata& metadata() const { return m_metadata; }
 
     bool operator==(const CSSProperty& other) const;
 
     DEFINE_INLINE_TRACE() { visitor->trace(m_value); }
 
 private:
     StylePropertyMetadata m_metadata;
-    RefPtrWillBeMember<CSSValue> m_value;
+    CSSValue m_value;
 };
 
 } // namespace blink
 
+// TODO(sashab): Shouldn't initialize CSSProperty with null, since it contains a CSSValue.
 WTF_ALLOW_MOVE_AND_INIT_WITH_MEM_FUNCTIONS(blink::CSSProperty);
 
 #endif // CSSProperty_h