Title: [213912] branches/safari-603-branch/Source/WebCore
- Revision
- 213912
- Author
- matthew_han...@apple.com
- Date
- 2017-03-14 10:13:47 -0700 (Tue, 14 Mar 2017)
Log Message
Roll out r212554 via r212893. rdar://problem/30812551
Modified Paths
Diff
Modified: branches/safari-603-branch/Source/WebCore/ChangeLog (213911 => 213912)
--- branches/safari-603-branch/Source/WebCore/ChangeLog 2017-03-14 17:13:43 UTC (rev 213911)
+++ branches/safari-603-branch/Source/WebCore/ChangeLog 2017-03-14 17:13:47 UTC (rev 213912)
@@ -1,3 +1,7 @@
+2017-03-02 Matthew Hanson <matthew_han...@apple.com>
+
+ Rollout r212893. rdar://problem/30812551
+
2017-02-22 Matthew Hanson <matthew_han...@apple.com>
rdar://problem/30657889
Modified: branches/safari-603-branch/Source/WebCore/dom/Document.cpp (213911 => 213912)
--- branches/safari-603-branch/Source/WebCore/dom/Document.cpp 2017-03-14 17:13:43 UTC (rev 213911)
+++ branches/safari-603-branch/Source/WebCore/dom/Document.cpp 2017-03-14 17:13:47 UTC (rev 213912)
@@ -2234,16 +2234,10 @@
FrameDestructionObserver::frameDestroyed();
}
-void Document::didBecomeCurrentDocumentInView()
-{
- ASSERT(view());
- if (!hasLivingRenderTree())
- createRenderTree();
-}
-
void Document::destroyRenderTree()
{
ASSERT(hasLivingRenderTree());
+ ASSERT(m_pageCacheState != InPageCache);
FrameView* frameView = frame()->document() == this ? frame()->view() : nullptr;
Modified: branches/safari-603-branch/Source/WebCore/dom/Document.h (213911 => 213912)
--- branches/safari-603-branch/Source/WebCore/dom/Document.h 2017-03-14 17:13:43 UTC (rev 213911)
+++ branches/safari-603-branch/Source/WebCore/dom/Document.h 2017-03-14 17:13:47 UTC (rev 213912)
@@ -566,7 +566,6 @@
void destroyRenderTree();
void disconnectFromFrame();
void prepareForDestruction();
- void didBecomeCurrentDocumentInView();
// Override ScriptExecutionContext methods to do additional work
bool shouldBypassMainWorldContentSecurityPolicy() const final;
Modified: branches/safari-603-branch/Source/WebCore/page/Frame.cpp (213911 => 213912)
--- branches/safari-603-branch/Source/WebCore/page/Frame.cpp 2017-03-14 17:13:43 UTC (rev 213911)
+++ branches/safari-603-branch/Source/WebCore/page/Frame.cpp 2017-03-14 17:13:47 UTC (rev 213912)
@@ -256,15 +256,8 @@
if (m_eventHandler)
m_eventHandler->clear();
- bool hadLivingRenderTree = m_doc ? m_doc->hasLivingRenderTree() : false;
- if (hadLivingRenderTree)
- m_doc->destroyRenderTree();
-
m_view = WTFMove(view);
- if (hadLivingRenderTree && m_view)
- m_doc->didBecomeCurrentDocumentInView();
-
// Only one form submission is allowed per view of a part.
// Since this part may be getting reused as a result of being
// pulled from the back/forward cache, reset this flag.
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
