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

Unified Diff: cc/layer_tree_host_impl.cc

Issue 11817011: cc: add RingBuffer class for timestamp storing in FrameRateCounter (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 11 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
Index: cc/layer_tree_host_impl.cc
diff --git a/cc/layer_tree_host_impl.cc b/cc/layer_tree_host_impl.cc
index 9d42698bae2a192a8e6884cd6011c818bd8040d1..5cb231184da8c4deb8682cf1e94653f166a205d8 100644
--- a/cc/layer_tree_host_impl.cc
+++ b/cc/layer_tree_host_impl.cc
@@ -885,7 +885,7 @@ void LayerTreeHostImpl::drawLayers(FrameData& frame)
// FIXME: use the frame begin time from the overall compositor scheduler.
// This value is currently inaccessible because it is up in Chromium's
// RenderWidget.
- m_fpsCounter->markBeginningOfFrame(base::TimeTicks::Now());
+ m_fpsCounter->saveTimeStamp(base::TimeTicks::Now());
if (m_debugState.showHudRects())
m_debugRectHistory->saveDebugRectsForCurrentFrame(rootLayer(), *frame.renderSurfaceLayerList, frame.occludingScreenSpaceRects, frame.nonOccludingScreenSpaceRects, m_debugState);
@@ -935,8 +935,6 @@ const RendererCapabilities& LayerTreeHostImpl::rendererCapabilities() const
bool LayerTreeHostImpl::swapBuffers()
{
DCHECK(m_renderer);
-
- m_fpsCounter->markEndOfFrame();
egraether 2013/01/09 02:04:04 I found this call had no real purpose and was lead
return m_renderer->swapBuffers();
}

Powered by Google App Engine
This is Rietveld 408576698