Preliminary paint property walk implementation for SPv2

third_party/WebKit/Source/core/layout/LayoutTestHelper.cpp

Index: third_party/WebKit/Source/core/layout/LayoutTestHelper.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutTestHelper.cpp b/third_party/WebKit/Source/core/layout/LayoutTestHelper.cpp
index 702089fc1520b4ab9e945607d00d7264a577ee71..84e47e59c616b2419562f462d01499aba4c7de7c 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTestHelper.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutTestHelper.cpp
@@ -43,5 +43,9 @@ void RenderingTest::SetUp()
     document().view()->updateAllLifecyclePhases();
 }
 
+void RenderingTest::TearDown()

[pdr., 2015/10/20 22:02:30]

Is this necessary? It should happen automatically as part of the regular test
runner lifecycle (either in SetUp or in the dtor).

[jbroman, 2015/10/20 23:50:13]

trchen answered this question when I asked it earlier, but if that ordering
issue still exists, it merits a comment here, please, if only so we stop asking.
:)

[trchen, 2015/10/21 06:16:19]

Still needed otherwise either an assertion goes off or we left dangling mapping.
Both sound terrible so let me add a comment. :)

+{
+    m_pageHolder = nullptr;
+}
 
 } // namespace blink