r100311 dropped a RefPtr that is very
authorjaphet@chromium.org <japhet@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 17 Nov 2011 23:35:22 +0000 (23:35 +0000)
committerjaphet@chromium.org <japhet@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 17 Nov 2011 23:35:22 +0000 (23:35 +0000)
helpful. Add it back.
https://bugs.webkit.org/show_bug.cgi?id=72647

Reviewed by Adam Barth.

http/tests/misc/xslt-bad-import.html should
stop crashing in full chromium builds.

* loader/cache/CachedResourceLoader.cpp:
(WebCore::CachedResourceLoader::loadDone):

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

Source/WebCore/ChangeLog
Source/WebCore/loader/cache/CachedResourceLoader.cpp

index 5576d72..09ade44 100644 (file)
@@ -1,3 +1,17 @@
+2011-11-17  Nate Chapin  <japhet@chromium.org>
+
+        r100311 dropped a RefPtr that is very
+        helpful. Add it back.
+        https://bugs.webkit.org/show_bug.cgi?id=72647
+
+        Reviewed by Adam Barth.
+
+        http/tests/misc/xslt-bad-import.html should
+        stop crashing in full chromium builds.
+
+        * loader/cache/CachedResourceLoader.cpp:
+        (WebCore::CachedResourceLoader::loadDone):
+
 2011-11-17  Adam Barth  <abarth@webkit.org>
 
         Refactor SecurityOrigin::create to be easier to understand
index 71d2f6b..490b99a 100644 (file)
@@ -653,6 +653,8 @@ void CachedResourceLoader::removeCachedResource(CachedResource* resource) const
 void CachedResourceLoader::loadDone()
 {
     m_loadFinishing = false;
+
+    RefPtr<Document> protect(m_document);
     if (frame())
         frame()->loader()->loadDone();
     performPostLoadActions();