Reviewed by Adele.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Jul 2006 05:55:15 +0000 (05:55 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Jul 2006 05:55:15 +0000 (05:55 +0000)
        - fix 308 Node leaks on the editing layout tests

        * dom/Document.cpp: (WebCore::Document::removedLastRef): Clear the markers once there
        are no real refs on the document, otherwise a self-only reference cycle w/ the document
        can occur.

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

WebCore/ChangeLog
WebCore/dom/Document.cpp

index 61318944842cf2fcd870c8fc1f3e7b742be4ffb8..2435bec3c13256f5a6fb2851df475641457b736e 100644 (file)
@@ -1,3 +1,13 @@
+2006-07-05  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Adele.
+
+        - fix 308 Node leaks on the editing layout tests
+
+        * dom/Document.cpp: (WebCore::Document::removedLastRef): Clear the markers once there
+        are no real refs on the document, otherwise a self-only reference cycle w/ the document
+        can occur.
+
 2006-07-05  Adele Peterson  <adele@apple.com>
 
         Reviewed by Maciej and Hyatt.
index ab8f6ca09038719ac19b6f1cc67f96cd55dfc59e..b9480075b3ce20a629214834f3b7a99da453fb13 100644 (file)
@@ -302,6 +302,8 @@ void Document::removedLastRef()
         m_titleElement = 0;
 
         removeAllChildren();
+
+        m_markers.clear();
     } else
         delete this;
 }