Unreviewed, rolling out r240896.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 3 Feb 2019 13:53:54 +0000 (13:53 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 3 Feb 2019 13:53:54 +0000 (13:53 +0000)
https://bugs.webkit.org/show_bug.cgi?id=194202

"Fixes leaks, but is probably not the correct fix." (Requested
by ddkilzer on #webkit).

Reverted changeset:

"Leak of WTF::StringImpl under SymbolImpl::createNullSymbol()
(48 bytes) in com.apple.WebKit.WebContent running layout
tests"
https://bugs.webkit.org/show_bug.cgi?id=193291
https://trac.webkit.org/changeset/240896

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

Source/WTF/ChangeLog
Source/WTF/wtf/text/SymbolImpl.h

index 3864715..13ca922 100644 (file)
@@ -1,3 +1,19 @@
+2019-02-03  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r240896.
+        https://bugs.webkit.org/show_bug.cgi?id=194202
+
+        "Fixes leaks, but is probably not the correct fix." (Requested
+        by ddkilzer on #webkit).
+
+        Reverted changeset:
+
+        "Leak of WTF::StringImpl under SymbolImpl::createNullSymbol()
+        (48 bytes) in com.apple.WebKit.WebContent running layout
+        tests"
+        https://bugs.webkit.org/show_bug.cgi?id=193291
+        https://trac.webkit.org/changeset/240896
+
 2019-02-02  David Kilzer  <ddkilzer@apple.com>
 
         Leak of WTF::StringImpl under SymbolImpl::createNullSymbol() (48 bytes) in com.apple.WebKit.WebContent running layout tests
index e082e1e..42ff036 100644 (file)
@@ -116,13 +116,6 @@ protected:
         ASSERT(StringImpl::tailOffset<StringImpl*>() == OBJECT_OFFSETOF(SymbolImpl, m_owner));
     }
 
-    ~SymbolImpl()
-    {
-        if (m_owner != StringImpl::empty())
-            m_owner->deref();
-        m_owner = nullptr;
-    }
-
     // The pointer to the owner string should be immediately following after the StringImpl layout,
     // since we would like to align the layout of SymbolImpl to the one of BufferSubstring StringImpl.
     StringImpl* m_owner;