Add assertion in Frame::setView() to make sure associated document has no living...
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 3 Mar 2017 17:51:23 +0000 (17:51 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 3 Mar 2017 17:51:23 +0000 (17:51 +0000)
https://bugs.webkit.org/show_bug.cgi?id=169134

Reviewed by Brent Fulgham.

Add assertion in Frame::setView() to make sure associated document has no living render tree.
This is not supposed to happen and r213311 fixed one case where this could happen. We need
to find out if it can still happen.

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

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

Source/WebCore/ChangeLog
Source/WebCore/page/Frame.cpp

index ba388f7..a9ceeec 100644 (file)
@@ -1,3 +1,17 @@
+2017-03-03  Chris Dumez  <cdumez@apple.com>
+
+        Add assertion in Frame::setView() to make sure associated document has no living render tree
+        https://bugs.webkit.org/show_bug.cgi?id=169134
+
+        Reviewed by Brent Fulgham.
+
+        Add assertion in Frame::setView() to make sure associated document has no living render tree.
+        This is not supposed to happen and r213311 fixed one case where this could happen. We need
+        to find out if it can still happen.
+
+        * page/Frame.cpp:
+        (WebCore::Frame::setView):
+
 2017-03-02  Sam Weinig  <sam@webkit.org>
 
         Rename htmlediting.h/cpp to Editing.h/cpp
index 73cba7d..40e387b 100644 (file)
@@ -256,6 +256,8 @@ void Frame::setView(RefPtr<FrameView>&& view)
         m_eventHandler->clear();
 
     bool hadLivingRenderTree = m_doc ? m_doc->hasLivingRenderTree() : false;
+    // Try and catch cases where this might still be happening after r213311.
+    ASSERT_WITH_SECURITY_IMPLICATION(!hadLivingRenderTree);
     if (hadLivingRenderTree)
         m_doc->destroyRenderTree();