Title: [89714] trunk/Source/WebCore
Revision
89714
Author
[email protected]
Date
2011-06-24 17:03:25 -0700 (Fri, 24 Jun 2011)

Log Message

        Rubber-stamped by Maciej Stachowiak.

        REGRESSION (r88984): Infinite recursion in DocumentLoader::detachFromFrame/stopLoading

        No new tests, as there is no known way to reproduce this (but we'll keep investigating, as
        the rollout will re-introduce the older less frequent crash).

        * loader/DocumentLoader.cpp: (WebCore::DocumentLoader::detachFromFrame): Rollout the fix
        for bug 62764.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (89713 => 89714)


--- trunk/Source/WebCore/ChangeLog	2011-06-24 23:50:39 UTC (rev 89713)
+++ trunk/Source/WebCore/ChangeLog	2011-06-25 00:03:25 UTC (rev 89714)
@@ -1,3 +1,15 @@
+2011-06-24  Alexey Proskuryakov  <[email protected]>
+
+        Rubber-stamped by Maciej Stachowiak.
+
+        REGRESSION (r88984): Infinite recursion in DocumentLoader::detachFromFrame/stopLoading
+
+        No new tests, as there is no known way to reproduce this (but we'll keep investigating, as
+        the rollout will re-introduce the older less frequent crash).
+
+        * loader/DocumentLoader.cpp: (WebCore::DocumentLoader::detachFromFrame): Rollout the fix
+        for bug 62764.
+
 2011-06-24  Tony Chang  <[email protected]>
 
         Reviewed by Ojan Vafai.

Modified: trunk/Source/WebCore/loader/DocumentLoader.cpp (89713 => 89714)


--- trunk/Source/WebCore/loader/DocumentLoader.cpp	2011-06-24 23:50:39 UTC (rev 89713)
+++ trunk/Source/WebCore/loader/DocumentLoader.cpp	2011-06-25 00:03:25 UTC (rev 89714)
@@ -400,10 +400,6 @@
 {
     ASSERT(m_frame);
 
-    // It never makes sense to have a document loader that is detached from its
-    // frame have any loads active, so go ahead and kill all the loads.
-    stopLoading();
-
 #if ENABLE(OFFLINE_WEB_APPLICATIONS)
     m_applicationCacheHost->setDOMApplicationCache(0);
 #endif
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to