Title: [155089] trunk/Source/WebCore
- Revision
- 155089
- Author
- [email protected]
- Date
- 2013-09-04 20:04:58 -0700 (Wed, 04 Sep 2013)
Log Message
Set "render tree being torn down" flag a bit earlier.
<https://webkit.org/b/120717>
Reviewed by Anders Carlsson.
Set the flag at the start of Document::detach() so everyone can get a chance
to avoid unnecessary work, not just renderers.
* dom/Document.cpp:
(WebCore::Document::detach):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (155088 => 155089)
--- trunk/Source/WebCore/ChangeLog 2013-09-05 02:53:16 UTC (rev 155088)
+++ trunk/Source/WebCore/ChangeLog 2013-09-05 03:04:58 UTC (rev 155089)
@@ -1,5 +1,18 @@
2013-09-04 Andreas Kling <[email protected]>
+ Set "render tree being torn down" flag a bit earlier.
+ <https://webkit.org/b/120717>
+
+ Reviewed by Anders Carlsson.
+
+ Set the flag at the start of Document::detach() so everyone can get a chance
+ to avoid unnecessary work, not just renderers.
+
+ * dom/Document.cpp:
+ (WebCore::Document::detach):
+
+2013-09-04 Andreas Kling <[email protected]>
+
Cached Page and Frame don't need to be ref-counted.
<https://webkit.org/b/120710>
Modified: trunk/Source/WebCore/dom/Document.cpp (155088 => 155089)
--- trunk/Source/WebCore/dom/Document.cpp 2013-09-05 02:53:16 UTC (rev 155088)
+++ trunk/Source/WebCore/dom/Document.cpp 2013-09-05 03:04:58 UTC (rev 155089)
@@ -2046,6 +2046,8 @@
ASSERT(attached());
ASSERT(!m_inPageCache);
+ TemporaryChange<bool> change(m_renderTreeBeingDestroyed, true);
+
#if ENABLE(POINTER_LOCK)
if (page())
page()->pointerLockController()->documentDetached(this);
@@ -2087,8 +2089,6 @@
m_focusedElement = 0;
m_activeElement = 0;
- TemporaryChange<bool> change(m_renderTreeBeingDestroyed, true);
-
if (m_documentElement)
Style::detachRenderTree(*m_documentElement);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
