Title: [89067] trunk/Source/WebCore
Revision
89067
Author
[email protected]
Date
2011-06-16 14:27:30 -0700 (Thu, 16 Jun 2011)

Log Message

2011-06-16  Abhishek Arya  <[email protected]>

        Reviewed by Adam Barth.

        RefPtr frame since it can get removed in
        FrameLoader::finishedParsing.
        https://bugs.webkit.org/show_bug.cgi?id=62812

        Tests: already tested by fast/parser/document-write-into-initial-document.html.

        * dom/Document.cpp:
        (WebCore::Document::finishedParsing):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (89066 => 89067)


--- trunk/Source/WebCore/ChangeLog	2011-06-16 21:20:52 UTC (rev 89066)
+++ trunk/Source/WebCore/ChangeLog	2011-06-16 21:27:30 UTC (rev 89067)
@@ -1,3 +1,16 @@
+2011-06-16  Abhishek Arya  <[email protected]>
+
+        Reviewed by Adam Barth.
+
+        RefPtr frame since it can get removed in
+        FrameLoader::finishedParsing.
+        https://bugs.webkit.org/show_bug.cgi?id=62812
+
+        Tests: already tested by fast/parser/document-write-into-initial-document.html.
+
+        * dom/Document.cpp:
+        (WebCore::Document::finishedParsing):
+
 2011-06-16  Sailesh Agrawal  <[email protected]>
 
         Reviewed by Mihai Parparita.

Modified: trunk/Source/WebCore/dom/Document.cpp (89066 => 89067)


--- trunk/Source/WebCore/dom/Document.cpp	2011-06-16 21:20:52 UTC (rev 89066)
+++ trunk/Source/WebCore/dom/Document.cpp	2011-06-16 21:27:30 UTC (rev 89067)
@@ -4140,7 +4140,7 @@
     if (!m_documentTiming.domContentLoadedEventEnd)
         m_documentTiming.domContentLoadedEventEnd = currentTime();
 
-    if (Frame* f = frame()) {
+    if (RefPtr<Frame> f = frame()) {
         // FrameLoader::finishedParsing() might end up calling Document::implicitClose() if all
         // resource loads are complete. HTMLObjectElements can start loading their resources from
         // post attach callbacks triggered by recalcStyle().  This means if we parse out an <object>
@@ -4152,7 +4152,7 @@
 
         f->loader()->finishedParsing();
 
-        InspectorInstrumentation::domContentLoadedEventFired(f, url());
+        InspectorInstrumentation::domContentLoadedEventFired(f.get(), url());
     }
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to