Remove histogram from GraphicsLayerUpdater::rebuildTree

Source/core/rendering/compositing/GraphicsLayerUpdater.cpp

 /*
  * Copyright (C) 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2014 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 20 matching lines @@
 #include "core/rendering/RenderLayer.h"
 #include "core/rendering/RenderLayerReflectionInfo.h"
 #include "core/rendering/RenderPart.h"
 #include "core/rendering/RenderView.h"
 #include "core/rendering/compositing/CompositedLayerMapping.h"
 #include "core/rendering/compositing/RenderLayerCompositor.h"
 #include "public/platform/Platform.h"
 
 namespace WebCore {
 
-bool shouldAppendLayer(const RenderLayer& layer)
+static bool shouldAppendLayer(const RenderLayer& layer)
 {
     if (!RuntimeEnabledFeatures::overlayFullscreenVideoEnabled())
         return true;
     Node* node = layer.renderer()->node();
     if (node && isHTMLMediaElement(*node) && toHTMLMediaElement(node)->isFullscreen())
         return false;
     return true;
 }
 
-GraphicsLayerUpdater::GraphicsLayerUpdater(RenderView& renderView)
-    : m_renderView(renderView)
-    , m_pixelsWithoutPromotingAllTransitions(0.0)
-    , m_pixelsAddedByPromotingAllTransitions(0.0)
+GraphicsLayerUpdater::GraphicsLayerUpdater()
 {
 }
 
 GraphicsLayerUpdater::~GraphicsLayerUpdater()
 {
 }
 
-void GraphicsLayerUpdater::rebuildTree(RenderLayer& layer, UpdateType updateType, GraphicsLayerVector& childLayersOfEnclosingLayer, int depth)
+void GraphicsLayerUpdater::rebuildTree(RenderLayer& layer, UpdateType updateType, GraphicsLayerVector& childLayersOfEnclosingLayer)
 {
     // Make the layer compositing if necessary, and set up clipping and content layers.
     // Note that we can only do work here that is independent of whether the descendant layers
     // 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();
 
     updateType = update(layer, updateType);
 
-    // Grab some stats for histograms.
-    if (hasCompositedLayerMapping) {
-        m_pixelsWithoutPromotingAllTransitions += layer.size().height() * layer.size().width();
-    } else {
-        if ((layer.renderer()->style()->transitionForProperty(CSSPropertyOpacity) ||
-             layer.renderer()->style()->transitionForProperty(CSSPropertyWebkitTransform)) &&
-            m_renderView.viewRect().intersects(layer.absoluteBoundingBox()))
-            m_pixelsAddedByPromotingAllTransitions += layer.size().height() * layer.size().width();
-    }
-
     // If this layer has a compositedLayerMapping, then that is where we place subsequent children GraphicsLayers.
     // Otherwise children continue to append to the child list of the enclosing layer.
     GraphicsLayerVector layerChildren;
     GraphicsLayerVector& childList = hasCompositedLayerMapping ? layerChildren : childLayersOfEnclosingLayer;
 
 #if !ASSERT_DISABLED
     LayerListMutationDetector mutationChecker(layer.stackingNode());
 #endif
 
     if (layer.stackingNode()->isStackingContainer()) {
         RenderLayerStackingNodeIterator iterator(*layer.stackingNode(), NegativeZOrderChildren);
         while (RenderLayerStackingNode* curNode = iterator.next())
-            rebuildTree(*curNode->layer(), updateType, childList, depth + 1);
+            rebuildTree(*curNode->layer(), updateType, childList);
 
         // If a negative z-order child is compositing, we get a foreground layer which needs to get parented.
         if (hasCompositedLayerMapping && currentCompositedLayerMapping->foregroundLayer())
             childList.append(currentCompositedLayerMapping->foregroundLayer());
     }
 
     RenderLayerStackingNodeIterator iterator(*layer.stackingNode(), NormalFlowChildren | PositiveZOrderChildren);
     while (RenderLayerStackingNode* curNode = iterator.next())
-        rebuildTree(*curNode->layer(), updateType, childList, depth + 1);
+        rebuildTree(*curNode->layer(), updateType, childList);
 
     if (hasCompositedLayerMapping) {
         bool parented = false;
         if (layer.renderer()->isRenderPart())
             parented = RenderLayerCompositor::parentFrameContentLayers(toRenderPart(layer.renderer()));
 
         if (!parented)
             currentCompositedLayerMapping->parentForSublayers()->setChildren(layerChildren);
 
         // If the layer has a clipping layer the overflow controls layers will be siblings of the clipping layer.
@@ skipping 11 matching lines @@
 
             if (GraphicsLayer* overflowControlLayer = currentCompositedLayerMapping->layerForScrollCorner()) {
                 overflowControlLayer->removeFromParent();
                 currentCompositedLayerMapping->parentForSublayers()->addChild(overflowControlLayer);
             }
         }
 
         if (shouldAppendLayer(layer))
             childLayersOfEnclosingLayer.append(currentCompositedLayerMapping->childForSuperlayers());
     }
-
-    if (!depth) {
-        int percentageIncreaseInPixels = static_cast<int>(m_pixelsAddedByPromotingAllTransitions / m_pixelsWithoutPromotingAllTransitions * 100);
-        blink::Platform::current()->histogramCustomCounts("Renderer.PixelIncreaseFromTransitions", percentageIncreaseInPixels, 0, 1000, 50);
-    }
 }
 
 // This just updates layer geometry without changing the hierarchy.
 void GraphicsLayerUpdater::updateRecursive(RenderLayer& layer, UpdateType updateType)
 {
     updateType = update(layer, updateType);
 
 #if !ASSERT_DISABLED
     LayerListMutationDetector mutationChecker(layer.stackingNode());
 #endif
@@ skipping 26 matching lines @@
     if (!layer.parent())
         layer.compositor()->updateRootLayerPosition();
 
     if (mapping->hasUnpositionedOverflowControlsLayers())
         layer.scrollableArea()->positionOverflowControls();
 
     return updateType;
 }
 
 } // namespace WebCore