Remove incorrect WTF::Vector traits for CSSProperty.
authorkling@webkit.org <kling@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Mar 2012 11:31:04 +0000 (11:31 +0000)
committerkling@webkit.org <kling@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Mar 2012 11:31:04 +0000 (11:31 +0000)
<http://webkit.org/b/53285>
<rdar://problem/10161286>

Reviewed by Antti Koivisto.

We shouldn't allow copying Vector<CSSProperty> contents around with memcpy()
as CSSProperty has a RefPtr member and this would muck up the refcount.
This was added to facilitate fast comparison of property vectors, and we no
longer use that, so there shouldn't be any performance concerns here.

* css/CSSProperty.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@109893 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/css/CSSProperty.h

index 9dc01fe..ed646a6 100644 (file)
@@ -1,3 +1,18 @@
+2012-03-06  Andreas Kling  <awesomekling@apple.com>
+
+        Remove incorrect WTF::Vector traits for CSSProperty.
+        <http://webkit.org/b/53285>
+        <rdar://problem/10161286>
+
+        Reviewed by Antti Koivisto.
+
+        We shouldn't allow copying Vector<CSSProperty> contents around with memcpy()
+        as CSSProperty has a RefPtr member and this would muck up the refcount.
+        This was added to facilitate fast comparison of property vectors, and we no
+        longer use that, so there shouldn't be any performance concerns here.
+
+        * css/CSSProperty.h:
+
 2012-03-06  Adam Barth  <abarth@webkit.org>
 
         Attempt to fix the Qt minimal build.
index 2674544..101a6fd 100644 (file)
@@ -68,9 +68,4 @@ public:
 
 } // namespace WebCore
 
-namespace WTF {
-    // Properties in Vector can be initialized with memset and moved using memcpy.
-    template<> struct VectorTraits<WebCore::CSSProperty> : SimpleClassVectorTraits { };
-}
-
 #endif // CSSProperty_h