Put transform tree building in DisplayItemPropertyTreeBuilder.

Source/core/compositing/DisplayListCompositingBuilder.cpp

Index: Source/core/compositing/DisplayListCompositingBuilder.cpp
diff --git a/Source/core/compositing/DisplayListCompositingBuilder.cpp b/Source/core/compositing/DisplayListCompositingBuilder.cpp
index 95a5baef84e315ddf1cd69c919089cd9df10f7f9..d8d3f45a77a456a0d08ba0f20621047579c06412 100644
--- a/Source/core/compositing/DisplayListCompositingBuilder.cpp
+++ b/Source/core/compositing/DisplayListCompositingBuilder.cpp
@@ -5,7 +5,7 @@
 #include "config.h"
 #include "core/compositing/DisplayListCompositingBuilder.h"
 
-#include "platform/graphics/paint/DisplayItemTransformTreeBuilder.h"
+#include "platform/graphics/paint/DisplayItemPropertyTreeBuilder.h"
 
 namespace blink {
 
@@ -14,10 +14,12 @@ void DisplayListCompositingBuilder::build(CompositedDisplayList& compositedDispl
     // TODO(pdr): Properly implement simple layer compositing here.
     // See: https://docs.google.com/document/d/1qF7wpO_lhuxUO6YXKZ3CJuXi0grcb5gKZJBBgnoTd0k/view
 
-    DisplayItemTransformTreeBuilder transformTreeBuilder;
+    DisplayItemPropertyTreeBuilder treeBuilder;
     for (const auto& displayItem : m_displayItemList.displayItems())
-        transformTreeBuilder.processDisplayItem(displayItem);
-    compositedDisplayList.transformTree = transformTreeBuilder.releaseTransformTree();
+        treeBuilder.processDisplayItem(displayItem);
+    compositedDisplayList.transformTree = treeBuilder.releaseTransformTree();
+    // TODO(pdr, jbroman): Also release other trees, and use range records to

[pdr., 2015/08/17 22:25:07]

Nit: TODO(jbroman)

We just use the person who added the todo now, but it doesn't mean you're on the
hook for this.

[jbroman, 2015/08/17 23:57:19]

That's not what the code style says, nor what I was told when I started at
Google.

https://www.chromium.org/blink/coding-style ->
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#TODO_Comments

says "TODOs should include the string TODO in all caps, followed by the name,
e-mail address, or other identifier of the person with the best context about
the problem referenced by the TODO."

My understanding is that it's supposed to be more "who knows what this means"
than "who added it" or "who should do it".

I can change it to just TODO(jbroman) if you prefer, but I'm unconvinced that
I'm only allowed to write TODO(jbroman).

[pdr., 2015/08/18 04:31:41]

I re-read the recent thread and you're correct. Disregard!

+    // construct simple layers.
 }
 
 } // namespace blink