update3dRenderingContext is 13.5% of GraphicsLayerUpdater::rebuildTree

Source/core/rendering/compositing/GraphicsLayerUpdater.cpp

Index: Source/core/rendering/compositing/GraphicsLayerUpdater.cpp
diff --git a/Source/core/rendering/compositing/GraphicsLayerUpdater.cpp b/Source/core/rendering/compositing/GraphicsLayerUpdater.cpp
index c27f6d0a28d91e94397ef718b1de47c3bcd8ac43..bbe95a2c44a1a5b4acb6f3ce8b459366254b227a 100644
--- a/Source/core/rendering/compositing/GraphicsLayerUpdater.cpp
+++ b/Source/core/rendering/compositing/GraphicsLayerUpdater.cpp
@@ -66,7 +66,6 @@ void GraphicsLayerUpdater::rebuildTree(RenderLayer& layer, UpdateType updateType
     // have been processed. computeCompositingRequirements() will already have done the repaint if necessary.
 
     layer.stackingNode()->updateLayerListsIfNeeded();
-    layer.update3dRenderingContext();
 
     const bool hasCompositedLayerMapping = layer.hasCompositedLayerMapping();
     CompositedLayerMappingPtr currentCompositedLayerMapping = layer.compositedLayerMapping();
@@ -159,6 +158,7 @@ void GraphicsLayerUpdater::updateRecursive(RenderLayer& layer, UpdateType update
 
 GraphicsLayerUpdater::UpdateType GraphicsLayerUpdater::update(RenderLayer& layer, UpdateType updateType)
 {
+

[ojan, 2014/03/25 01:36:00]

Did you mean to add this?

[abarth-chromium, 2014/03/25 02:31:38]

Nope!  That was from an earlier iteration where I tried to move the work here.

     if (!layer.hasCompositedLayerMapping())
         return updateType;