Add mutableComponent() interface to InterpolationValue

Source/core/animation/InterpolationValue.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef InterpolationValue_h
 #define InterpolationValue_h
 
 #include "core/animation/InterpolableValue.h"
 #include "core/animation/NonInterpolableValue.h"
 #include "platform/heap/Handle.h"
 
 namespace blink {
 
 class InterpolationType;
 
+struct InterpolationComponentValue {
+    ALLOW_ONLY_INLINE_ALLOCATION();
+
+    InterpolationComponentValue(PassOwnPtr<InterpolableValue> interpolableValue = nullptr, PassRefPtr<NonInterpolableValue> nonInterpolableValue = nullptr)
+        : interpolableValue(interpolableValue)
+        , nonInterpolableValue(nonInterpolableValue)
+    { }
+
+    OwnPtr<InterpolableValue> interpolableValue;
+    RefPtr<NonInterpolableValue> nonInterpolableValue;
+};
+
 class InterpolationValue {
 public:
     static PassOwnPtr<InterpolationValue> create(const InterpolationType& type, PassOwnPtr<InterpolableValue> interpolableValue, PassRefPtrWillBeRawPtr<NonInterpolableValue> nonInterpolableValue = nullptr)
     {
         return adoptPtr(new InterpolationValue(type, interpolableValue, nonInterpolableValue));
     }
 
     PassOwnPtr<InterpolationValue> clone() const
     {
-        return create(m_type, m_interpolableValue->clone(), m_nonInterpolableValue);
+        return create(m_type, m_component.interpolableValue->clone(), m_component.nonInterpolableValue);
     }
 
     const InterpolationType& type() const { return m_type; }
-    const InterpolableValue& interpolableValue() const { return *m_interpolableValue; }
-    InterpolableValue& interpolableValue() { return *m_interpolableValue; }
-    const NonInterpolableValue* nonInterpolableValue() const { return m_nonInterpolableValue.get(); }
+    const InterpolableValue& interpolableValue() const { return *m_component.interpolableValue; }
+    const NonInterpolableValue* nonInterpolableValue() const { return m_component.nonInterpolableValue.get(); }
+
+    InterpolationComponentValue& mutableComponent() { return m_component; }

[dstockwell, 2015/09/14 00:24:36]

The component naming doesn't help here, this sounds like "give me the mutable
part of the larger structure".

[alancutter (OOO until 2018), 2015/09/14 00:37:39]

This description is an accurate interpretation of the data structure and
purpose.

 
 private:
     InterpolationValue(const InterpolationType& type, PassOwnPtr<InterpolableValue> interpolableValue, PassRefPtrWillBeRawPtr<NonInterpolableValue> nonInterpolableValue = nullptr)
         : m_type(type)
-        , m_interpolableValue(interpolableValue)
-        , m_nonInterpolableValue(nonInterpolableValue)
+        , m_component(interpolableValue, nonInterpolableValue)
     {
-        ASSERT(this->m_interpolableValue);
+        ASSERT(m_component.interpolableValue);
     }
 
     const InterpolationType& m_type;
-    OwnPtr<InterpolableValue> m_interpolableValue;
-    RefPtrWillBePersistent<NonInterpolableValue> m_nonInterpolableValue;
-
-    friend class InterpolationType;
+    InterpolationComponentValue m_component;
 };
 
 } // namespace blink
 
 #endif // InterpolationValue_h