2009-03-27 Darin Adler <darin@apple.com>

[WebKit-https.git] / WebCore / dom / Document.cpp

diff --git a/WebCore/dom/Document.cpp b/WebCore/dom/Document.cpp
index 5fb117f..782dd36 100644 (file)
--- a/WebCore/dom/Document.cpp
+++ b/WebCore/dom/Document.cpp
@@ -1318,10 +1318,11 @@ void Document::detach()
         render->destroy();
     
     // This is required, as our Frame might delete itself as soon as it detaches
-    // us.  However, this violates Node::detach() symantics, as it's never
-    // possible to re-attach.  Eventually Document::detach() should be renamed
-    // or this call made explicit in each of the callers of Document::detach().
-    clearFramePointer();
+    // us. However, this violates Node::detach() symantics, as it's never
+    // possible to re-attach. Eventually Document::detach() should be renamed,
+    // or this setting of the frame to 0 could be made explicit in each of the
+    // callers of Document::detach().
+    m_frame = 0;
     
     if (m_renderArena) {
         delete m_renderArena;
@@ -1329,11 +1330,6 @@ void Document::detach()
     }
 }
 
-void Document::clearFramePointer()
-{
-    m_frame = 0;
-}
-
 void Document::removeAllEventListenersFromAllNodes()
 {
     size_t size = m_windowEventListeners.size();