2010-11-19 Daniel Bates <dbates@rim.com>
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Nov 2010 20:08:20 +0000 (20:08 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Nov 2010 20:08:20 +0000 (20:08 +0000)
        Reviewed by Antonio Gomes.

        Replace null checks of newParent in Frame::transferChildFrameToNewDocument() with ASSERT
        https://bugs.webkit.org/show_bug.cgi?id=49489

        Substitute ASSERT(newParent) for null-checks for newParent in
        Frame::transferChildFrameToNewDocument() since newParent is
        never null.

        When Frame::transferChildFrameToNewDocument() is called the child
        frame's owner element has already been transferred to the new
        document D and D is attached (i.e. has a frame).

        * page/Frame.cpp:
        (WebCore::Frame::transferChildFrameToNewDocument):

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

WebCore/ChangeLog
WebCore/page/Frame.cpp

index 0dc7e6e..9f79db7 100644 (file)
@@ -1,3 +1,21 @@
+2010-11-19  Daniel Bates  <dbates@rim.com>
+
+        Reviewed by Antonio Gomes.
+
+        Replace null checks of newParent in Frame::transferChildFrameToNewDocument() with ASSERT
+        https://bugs.webkit.org/show_bug.cgi?id=49489
+
+        Substitute ASSERT(newParent) for null-checks for newParent in
+        Frame::transferChildFrameToNewDocument() since newParent is
+        never null.
+
+        When Frame::transferChildFrameToNewDocument() is called the child
+        frame's owner element has already been transferred to the new
+        document D and D is attached (i.e. has a frame).
+
+        * page/Frame.cpp:
+        (WebCore::Frame::transferChildFrameToNewDocument):
+
 2010-11-19  Anders Carlsson  <andersca@apple.com>
 
         Reviewed by Sam Weinig.
index 970078e..5030c72 100644 (file)
@@ -711,10 +711,11 @@ void Frame::transferChildFrameToNewDocument()
 {
     ASSERT(m_ownerElement);
     Frame* newParent = m_ownerElement->document()->frame();
+    ASSERT(newParent);
     bool didTransfer = false;
 
     // Switch page.
-    Page* newPage = newParent ? newParent->page() : 0;
+    Page* newPage = newParent->page();
     Page* oldPage = m_page;
     if (m_page != newPage) {
         if (m_page) {
@@ -732,10 +733,8 @@ void Frame::transferChildFrameToNewDocument()
         didTransfer = true;
     }
 
-    if (newParent) {
-        // Update the frame tree.
-        didTransfer = newParent->tree()->transferChild(this);
-    }
+    // Update the frame tree.
+    didTransfer = newParent->tree()->transferChild(this);
 
     // Avoid unnecessary calls to client and frame subtree if the frame ended
     // up on the same page and under the same parent frame.