Changeset 220484 in webkit

Timestamp:

Aug 9, 2017 3:08:35 PM (7 years ago)

Author:

commit-queue@webkit.org

Message:

Unreviewed, rolling out r219334.
​https://bugs.webkit.org/show_bug.cgi?id=175398

Caused bug 175023 (Requested by ap on #webkit).

Reverted changeset:

"[SVG] Leak in SVGAnimatedListPropertyTearOff"
​https://bugs.webkit.org/show_bug.cgi?id=172545
​http://trac.webkit.org/changeset/219334

Location:

trunk

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/svg/animations/animation-leak-list-property-instances-expected.txt (deleted)
• LayoutTests/svg/animations/animation-leak-list-property-instances.html (deleted)
• Source/WebCore/ChangeLog (modified) (1 diff)
• Source/WebCore/svg/properties/SVGAnimatedListPropertyTearOff.h (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2017-08-09  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r219334.
+        https://bugs.webkit.org/show_bug.cgi?id=175398
+
+        Caused bug 175023 (Requested by ap on #webkit).
+
+        Reverted changeset:
+
+        "[SVG] Leak in SVGAnimatedListPropertyTearOff"
+        https://bugs.webkit.org/show_bug.cgi?id=172545
+        http://trac.webkit.org/changeset/219334
+
 2017-08-09  Daniel Bates  <dabates@apple.com>
 

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-08-09  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r219334.
+        https://bugs.webkit.org/show_bug.cgi?id=175398
+
+        Caused bug 175023 (Requested by ap on #webkit).
+
+        Reverted changeset:
+
+        "[SVG] Leak in SVGAnimatedListPropertyTearOff"
+        https://bugs.webkit.org/show_bug.cgi?id=172545
+        http://trac.webkit.org/changeset/219334
+
 2017-08-09  Dave Hyatt  <hyatt@apple.com>
 

trunk/Source/WebCore/svg/properties/SVGAnimatedListPropertyTearOff.h

@@ -74 +74 @@
         else if (&property == m_animVal)
             m_animVal = nullptr;
-        if (!m_baseVal && !m_animVal)
-            detachListWrappers(m_values.size());
     }