Remove redundant size member from InterpolableList

third_party/WebKit/Source/core/animation/InterpolableValue.h

 // Copyright 2014 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 InterpolableValue_h
 #define InterpolableValue_h
 
 #include "core/CoreExport.h"
 #include "core/animation/animatable/AnimatableValue.h"
 #include "platform/heap/Handle.h"
@@ skipping 104 matching lines @@
     }
 
     static std::unique_ptr<InterpolableList> create(size_t size)
     {
         return wrapUnique(new InterpolableList(size));
     }
 
     bool isList() const final { return true; }
     void set(size_t position, std::unique_ptr<InterpolableValue> value)
     {
-        DCHECK_LT(position, m_size);
         m_values[position] = std::move(value);
     }
     const InterpolableValue* get(size_t position) const
     {
-        DCHECK_LT(position, m_size);
         return m_values[position].get();
     }
     std::unique_ptr<InterpolableValue>& getMutable(size_t position)
     {
-        DCHECK_LT(position, m_size);

[alancutter (OOO until 2018), 2016/08/25 02:03:53]

These checks are already performed by Vector::at() as RELEASE_ASSERTs.

         return m_values[position];
     }
-    size_t length() const { return m_size; }
+    size_t length() const { return m_values.size(); }
     bool equals(const InterpolableValue& other) const final;
     std::unique_ptr<InterpolableValue> clone() const final { return create(*this); }
     std::unique_ptr<InterpolableValue> cloneAndZero() const final;
     void scale(double scale) final;
     void scaleAndAdd(double scale, const InterpolableValue& other) final;
 
 private:
     void interpolate(const InterpolableValue& to, const double progress, InterpolableValue& result) const final;
     explicit InterpolableList(size_t size)
-        : m_size(size)
-        , m_values(m_size)
+        : m_values(size)
     {
     }
 
     InterpolableList(const InterpolableList& other)
-        : m_size(other.m_size)
-        , m_values(m_size)
+        : m_values(other.length())
     {
-        for (size_t i = 0; i < m_size; i++)
+        for (size_t i = 0; i < length(); i++)
             set(i, other.m_values[i]->clone());
     }
 
-    size_t m_size;
     Vector<std::unique_ptr<InterpolableValue>> m_values;
 };
 
 // FIXME: Remove this when we can.
 class InterpolableAnimatableValue : public InterpolableValue {
 public:
     static std::unique_ptr<InterpolableAnimatableValue> create(PassRefPtr<AnimatableValue> value)
     {
         return wrapUnique(new InterpolableAnimatableValue(value));
     }
@@ skipping 17 matching lines @@
 };
 
 DEFINE_TYPE_CASTS(InterpolableNumber, InterpolableValue, value, value->isNumber(), value.isNumber());
 DEFINE_TYPE_CASTS(InterpolableBool, InterpolableValue, value, value->isBool(), value.isBool());
 DEFINE_TYPE_CASTS(InterpolableList, InterpolableValue, value, value->isList(), value.isList());
 DEFINE_TYPE_CASTS(InterpolableAnimatableValue, InterpolableValue, value, value->isAnimatableValue(), value.isAnimatableValue());
 
 } // namespace blink
 
 #endif