Reviewed by Maciej's rubber stamp
authorbeidson <beidson@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Sep 2006 23:44:33 +0000 (23:44 +0000)
committerbeidson <beidson@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Sep 2006 23:44:33 +0000 (23:44 +0000)
        Exact same fix I just made, but in the other method I horked up
        Also added a reference to the bugzilla in comments

        * page/Frame.cpp:
        (WebCore::Frame::endIfNotLoading):
        (WebCore::Frame::stop):

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

WebCore/ChangeLog
WebCore/page/Frame.cpp

index 7e60e3f2223d0b7ebd66ce9dfac4316c4a507cb9..5aa391522abd505952d7fc49f29e9e80c5e65484 100644 (file)
@@ -1,3 +1,14 @@
+2006-09-14  Brady Eidson  <beidson@apple.com>
+
+        Reviewed by Maciej's rubber stamp
+
+        Exact same fix I just made, but in the other method I horked up
+        Also added a reference to the bugzilla in comments
+
+        * page/Frame.cpp:
+        (WebCore::Frame::endIfNotLoading):
+        (WebCore::Frame::stop):
+
 2006-09-14  Brady Eidson  <beidson@apple.com>
 
         Reviewed by Maciej
index c99a12a8ad42925da002a10ba2cb94eaa56acc39..0d59c95de8239ad113d70e4149e4bd4014d87d6a 100644 (file)
@@ -728,6 +728,7 @@ void Frame::end()
 
 void Frame::endIfNotLoading()
 {
+    // http://bugzilla.opendarwin.org/show_bug.cgi?id=10854
     // The frame's last ref may be remove and it be deleted by checkCompleted(), 
     // so we'll add a protective refcount
     RefPtr<Frame> protector(this);
@@ -779,6 +780,11 @@ void Frame::endIfNotLoading()
 
 void Frame::stop()
 {
+    // http://bugzilla.opendarwin.org/show_bug.cgi?id=10854
+    // The frame's last ref may be remove and it be deleted by checkCompleted(), 
+    // so we'll add a protective refcount
+    RefPtr<Frame> protector(this);
+    
     if (d->m_doc) {
         if (d->m_doc->tokenizer())
             d->m_doc->tokenizer()->stopParsing();