Slimming Paint phase 2 compositing algorithm plumbing

Source/platform/graphics/ContentLayerDelegate.cpp

Index: Source/platform/graphics/ContentLayerDelegate.cpp
diff --git a/Source/platform/graphics/ContentLayerDelegate.cpp b/Source/platform/graphics/ContentLayerDelegate.cpp
index 657ecb92854104a0c810de448a4d56e21085ef67..4a9f97323dbfa20ee1379a53f0a0eda6cb4f8d75 100644
--- a/Source/platform/graphics/ContentLayerDelegate.cpp
+++ b/Source/platform/graphics/ContentLayerDelegate.cpp
@@ -90,6 +90,14 @@ void ContentLayerDelegate::paintContents(
 
     ASSERT(RuntimeEnabledFeatures::slimmingPaintEnabled());
 
+    if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
+        if (m_painter->displayItemList()) {

[Xianzhu, 2015/08/13 17:03:58]

Add DisplayItemList::appendToWebDisplayItemList() for v2?

[pdr., 2015/08/13 20:55:00]

I'm afraid I don't understand. Can you rephrase your question?

[Xianzhu, 2015/08/14 17:22:59]

I meant to extract line 95-96 as DisplayItemList::appendToWebDisplayItemList()
(which can be separated from
DisplayItemList::commitNewDisplayItemsAndAppendToWebDisplayItemList()). This
seems to be used by temporary code only for now, but will
DisplayListCompositingBuilder need this function? 

[pdr., 2015/08/15 02:35:04]

Ahh, good idea. I've separated this out as you suggested. We'll probably use it
in followup patches.

+            for (auto& displayItem : m_painter->displayItemList()->displayItems())
+                displayItem.appendToWebDisplayItemList(webDisplayItemList);
+        }
+        return;
+    }

[chrishtr, 2015/08/13 17:28:32]

Maybe we should just skip this glue code, which is temporary anyway.

[pdr., 2015/08/13 20:55:00]

Added a comment that this code is temporary and just here to make the browser
usable. Without it, we'll crash due to committing new display items twice.

+
     DisplayItemList* displayItemList = m_painter->displayItemList();
     ASSERT(displayItemList);
     displayItemList->setDisplayItemConstructionIsDisabled(