Introduce TerminatedArray<T> type to make data-structure used by RuleSet visible at the type level.

Source/core/css/RuleSet.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011. All rights reserved.
@@ skipping 107 matching lines @@
     }
     return PropertyWhitelistNone;
 }
 
 namespace {
 
 // FIXME: Should we move this class to WTF?
 template<typename T>
 class TerminatedArrayBuilder {
 public:
-    explicit TerminatedArrayBuilder(PassOwnPtr<T> array)
+    explicit TerminatedArrayBuilder(PassOwnPtr<TerminatedArray<T> > array)
         : m_array(array)
         , m_count(0)
         , m_capacity(0)
     {
         if (!m_array)
             return;
-        for (T* item = m_array.get(); !item->isLastInArray(); ++item)
-            ++m_count;
-        ++m_count; // To count the last item itself.
-        m_capacity = m_count;
+        m_capacity = m_count = m_array->size();
     }
 
     void grow(size_t count)
     {
         ASSERT(count);
         if (!m_array) {
             ASSERT(!m_count);
             ASSERT(!m_capacity);
             m_capacity = count;
-            m_array = adoptPtr(static_cast<T*>(fastMalloc(m_capacity * sizeof(T))));
+            m_array = TerminatedArray<T>::create(m_capacity);
             return;
         }
         m_capacity += count;
-        m_array = adoptPtr(static_cast<T*>(fastRealloc(m_array.leakPtr(), m_capacity * sizeof(T))));
-        m_array.get()[m_count - 1].setLastInArray(false);
+        m_array = m_array.leakPtr()->realloc(m_capacity);
+        m_array->at(m_count - 1).setLastInArray(false);
     }
 
     void append(const T& item)
     {
         RELEASE_ASSERT(m_count < m_capacity);
         ASSERT(!item.isLastInArray());
-        m_array.get()[m_count++] = item;
+        m_array->at(m_count++) = item;
     }
 
-    PassOwnPtr<T> release()
+    PassOwnPtr<TerminatedArray<T> > release()
     {
         RELEASE_ASSERT(m_count == m_capacity);
         if (m_array)
-            m_array.get()[m_count - 1].setLastInArray(true);
+            m_array->at(m_count - 1).setLastInArray(true);
         assertValid();
         return m_array.release();
     }
 
 private:
 #ifndef NDEBUG
     void assertValid()
     {
         for (size_t i = 0; i < m_count; ++i) {
             bool isLastInArray = (i + 1 == m_count);
-            ASSERT(m_array.get()[i].isLastInArray() == isLastInArray);
+            ASSERT(m_array->at(i).isLastInArray() == isLastInArray);
         }
     }
 #else
     void assertValid() { }
 #endif
 
-    OwnPtr<T> m_array;
+    OwnPtr<TerminatedArray<T> > m_array;
     size_t m_count;
     size_t m_capacity;
 };
 
 }
 
 RuleData::RuleData(StyleRule* rule, unsigned selectorIndex, unsigned position, AddRuleFlags addRuleFlags)
     : m_rule(rule)
     , m_selectorIndex(selectorIndex)
     , m_isLastInArray(false)
@@ skipping 237 matching lines @@
 #ifndef NDEBUG
 
 void RuleSet::show()
 {
     for (Vector<RuleData>::const_iterator it = m_allRules.begin(); it != m_allRules.end(); ++it)
         it->selector().show();
 }
 #endif
 
 } // namespace WebCore