ASSERT on renderer lifecycle

Source/core/dom/Node.cpp

Index: Source/core/dom/Node.cpp
diff --git a/Source/core/dom/Node.cpp b/Source/core/dom/Node.cpp
index 338aa5734b2ed6ec461f18cb4d2f1241a99d82d4..24d7fcf02e502e9249a73e4e918d02c4fe7bd4cf 100644
--- a/Source/core/dom/Node.cpp
+++ b/Source/core/dom/Node.cpp
@@ -995,6 +995,10 @@ bool Node::inDetach() const
 
 void Node::detach(const AttachContext& context)
 {
+    // FIXME: We should not modify the structure of the render tree during
+    // layout. crbug.com/370463

[esprehn, 2014/05/08 05:41:25]

This doesn't need a fix me, it's correct to modify the render tree inside
::detach.

[dsinclair, 2014/05/08 19:32:11]

Done.

+    DeprecatedDisableModifyRenderTreeStructureAsserts disabler;
+
 #ifndef NDEBUG
     ASSERT(!detachingNode);
     detachingNode = this;