Changeset 143054 in webkit

Timestamp:

Feb 15, 2013 2:47:35 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

Unreviewed, rolling out r143044.
​http://trac.webkit.org/changeset/143044
​https://bugs.webkit.org/show_bug.cgi?id=109974

broke windows build (Requested by kling on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2013-02-15

• dom/DocumentSharedObjectPool.cpp:

(WebCore::DocumentSharedObjectPool::cachedShareableElementDataWithAttributes):

• dom/Element.cpp:

(WebCore::sizeForShareableElementDataWithAttributeCount):
(WebCore::ShareableElementData::ShareableElementData):
(WebCore::ShareableElementData::~ShareableElementData):
(WebCore::UniqueElementData::UniqueElementData):

• dom/Element.h:

(WebCore::ShareableElementData::immutableAttributeArray):
(ShareableElementData):
(WebCore::ElementData::attributeItem):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• dom/DocumentSharedObjectPool.cpp (modified) (1 diff)
• dom/Element.cpp (modified) (5 diffs)
• dom/Element.h (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-02-15  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r143044.
+        http://trac.webkit.org/changeset/143044
+        https://bugs.webkit.org/show_bug.cgi?id=109974
+
+        broke windows build (Requested by kling on #webkit).
+
+        * dom/DocumentSharedObjectPool.cpp:
+        (WebCore::DocumentSharedObjectPool::cachedShareableElementDataWithAttributes):
+        * dom/Element.cpp:
+        (WebCore::sizeForShareableElementDataWithAttributeCount):
+        (WebCore::ShareableElementData::ShareableElementData):
+        (WebCore::ShareableElementData::~ShareableElementData):
+        (WebCore::UniqueElementData::UniqueElementData):
+        * dom/Element.h:
+        (WebCore::ShareableElementData::immutableAttributeArray):
+        (ShareableElementData):
+        (WebCore::ElementData::attributeItem):
+
 2013-02-15  Adam Barth  <abarth@webkit.org>
 

trunk/Source/WebCore/dom/DocumentSharedObjectPool.cpp

@@ -87 +87 @@
         return elementData.release();
 
-    cacheIterator->value = adoptPtr(new ShareableElementDataCacheEntry(ShareableElementDataCacheKey(elementData->m_attributeArray, elementData->length()), elementData));
+    cacheIterator->value = adoptPtr(new ShareableElementDataCacheEntry(ShareableElementDataCacheKey(elementData->immutableAttributeArray(), elementData->length()), elementData));
 
     return elementData.release();

trunk/Source/WebCore/dom/Element.cpp

@@ -2864 +2864 @@
 static size_t sizeForShareableElementDataWithAttributeCount(unsigned count)
 {
-    return sizeof(ShareableElementData) + sizeof(Attribute) * count;
+    return sizeof(ShareableElementData) - sizeof(void*) + sizeof(Attribute) * count;
 }
 
@@ -2882 +2882 @@
 {
     for (unsigned i = 0; i < m_arraySize; ++i)
-        new (&m_attributeArray[i]) Attribute(attributes[i]);
+        new (&reinterpret_cast<Attribute*>(&m_attributeArray)[i]) Attribute(attributes[i]);
 }
 
@@ -2888 +2888 @@
 {
     for (unsigned i = 0; i < m_arraySize; ++i)
-        m_attributeArray[i].~Attribute();
+        (reinterpret_cast<Attribute*>(&m_attributeArray)[i]).~Attribute();
 }
 
@@ -2902 +2902 @@
 
     for (unsigned i = 0; i < m_arraySize; ++i)
-        new (&m_attributeArray[i]) Attribute(other.m_attributeVector.at(i));
+        new (&reinterpret_cast<Attribute*>(&m_attributeArray)[i]) Attribute(other.m_attributeVector.at(i));
 }
 
@@ -2940 +2940 @@
     m_attributeVector.reserveCapacity(other.length());
     for (unsigned i = 0; i < other.length(); ++i)
-        m_attributeVector.uncheckedAppend(other.m_attributeArray[i]);
+        m_attributeVector.uncheckedAppend(other.immutableAttributeArray()[i]);
 }
 

trunk/Source/WebCore/dom/Element.h

@@ -120 +120 @@
 };
 
-#if COMPILER(MSVC)
-#pragma warning(push)
-#pragma warning(disable: 4200) // Disable "zero-sized array in struct/union" warning
-#endif
-
 class ShareableElementData : public ElementData {
 public:
     static PassRefPtr<ShareableElementData> createWithAttributes(const Vector<Attribute>&);
 
+    const Attribute* immutableAttributeArray() const { return reinterpret_cast<const Attribute*>(&m_attributeArray); }
+
     explicit ShareableElementData(const Vector<Attribute>&);
     explicit ShareableElementData(const UniqueElementData&);
     ~ShareableElementData();
 
-    Attribute m_attributeArray[0];
+    void* m_attributeArray;
 };
-
-#if COMPILER(MSVC)
-#pragma warning(pop)
-#endif
 
 class UniqueElementData : public ElementData {
@@ -1042 +1035 @@
     if (m_isUnique)
         return &static_cast<const UniqueElementData*>(this)->m_attributeVector.at(index);
-    return &static_cast<const ShareableElementData*>(this)->m_attributeArray[index];
+    return &static_cast<const ShareableElementData*>(this)->immutableAttributeArray()[index];
 }