Reviewed by Adam Roben.

        Add an ASSERT to better demonstrate the cause of the crash in
        https://bugs.webkit.org/show_bug.cgi?id=23736

        * loader/DocLoader.cpp:
        (WebCore::DocLoader::~DocLoader):

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

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 5ac656e3d82e9ecf2669dc8613e3e5991930ebd7..d265bc32b431f5d47ff45bcef068b09db3d984c8 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,13 @@
+2009-03-04  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by Adam Roben.
+
+        Add an ASSERT to better demonstrate the cause of the crash in
+        https://bugs.webkit.org/show_bug.cgi?id=23736
+
+        * loader/DocLoader.cpp:
+        (WebCore::DocLoader::~DocLoader):
+
 2009-03-04  David Levin  <levin@chromium.org>
 
         Reviewed by Alexey Proskuryakov.

diff --git a/WebCore/loader/DocLoader.cpp b/WebCore/loader/DocLoader.cpp
index 1c5f0b72443921ac25a1d00be1ae0d6697022973..007dffa34899de9f4d78cfd415d349b4de434acf 100644 (file)
--- a/WebCore/loader/DocLoader.cpp
+++ b/WebCore/loader/DocLoader.cpp
@@ -65,6 +65,9 @@ DocLoader::~DocLoader()
     for (DocumentResourceMap::iterator it = m_documentResources.begin(); it != end; ++it)
         it->second->setDocLoader(0);
     m_cache->removeDocLoader(this);
+
+    // Make sure no requests still point to this DocLoader
+    ASSERT(m_requestCount == 0);
 }
 
 Frame* DocLoader::frame() const