Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1549)

Unified Diff: Source/core/rendering/RenderLayerCompositor.cpp

Issue 23458035: UMA stats instrumentation for Renderer.AcceleratedFixedRootBackground (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/rendering/RenderLayerCompositor.cpp
diff --git a/Source/core/rendering/RenderLayerCompositor.cpp b/Source/core/rendering/RenderLayerCompositor.cpp
index e7b5a0f21a3d37333f34d10368c69e0914d43ea4..ca69154269837eb9da2c96fe8b1737e383ef9467 100644
--- a/Source/core/rendering/RenderLayerCompositor.cpp
+++ b/Source/core/rendering/RenderLayerCompositor.cpp
@@ -1193,6 +1193,13 @@ void RenderLayerCompositor::frameViewDidChangeSize()
}
}
+enum AcceleratedFixedRootBackgroundHistogramBuckets {
+ ScrolledMainFrameBucket = 0,
+ ScrolledMainFrameWithAcceleratedFixedRootBackground = 1,
+ ScrolledMainFrameWithUnacceleratedFixedRootBackground = 2,
+ AcceleratedFixedRootBackgroundHistogramMax = 3
+};
+
void RenderLayerCompositor::frameViewDidScroll()
{
FrameView* frameView = m_renderView->frameView();
@@ -1216,6 +1223,20 @@ void RenderLayerCompositor::frameViewDidScroll()
m_scrollLayer->setPosition(-frameView->minimumScrollPosition());
else
m_scrollLayer->setPosition(-scrollPosition);
+
+
+ HistogramSupport::histogramEnumeration("Renderer.AcceleratedFixedRootBackground",
+ ScrolledMainFrameBucket,
+ AcceleratedFixedRootBackgroundHistogramMax);
+
+ if (!m_renderView->rootBackgroundIsEntirelyFixed())
hartmanng 2013/09/09 14:31:26 I remember you mentioned that pages can have parti
+ return;
+
+ HistogramSupport::histogramEnumeration("Renderer.AcceleratedFixedRootBackground",
+ !!fixedRootBackgroundLayer()
+ ? ScrolledMainFrameWithAcceleratedFixedRootBackground
+ : ScrolledMainFrameWithUnacceleratedFixedRootBackground,
+ AcceleratedFixedRootBackgroundHistogramMax);
}
void RenderLayerCompositor::frameViewDidLayout()
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698