cc: add rasterize time to continuous painting graph data in impl-side-painting

cc/layer_tree_host_impl.cc

 // Copyright 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/layer_tree_host_impl.h"
 
 #include <algorithm>
 
 #include "base/basictypes.h"
 #include "base/debug/trace_event.h"
@@ skipping 130 matching lines @@
     , m_wheelScrolling(false)
     , m_settings(settings)
     , m_deviceScaleFactor(1)
     , m_visible(true)
     , m_managedMemoryPolicy(PrioritizedResourceManager::defaultMemoryAllocationLimit(),
                             ManagedMemoryPolicy::CUTOFF_ALLOW_EVERYTHING,
                             0,
                             ManagedMemoryPolicy::CUTOFF_ALLOW_NOTHING)
     , m_pinchGestureActive(false)
     , m_fpsCounter(FrameRateCounter::create(m_proxy->hasImplThread()))
-    , m_paintTimeCounter(PaintTimeCounter::create())

[nduca, 2013/02/13 08:12:21]

why do you defer the creation of the counter? It seems a bit risky to me

     , m_memoryHistory(MemoryHistory::create())
     , m_debugRectHistory(DebugRectHistory::create())
     , m_numImplThreadScrolls(0)
     , m_numMainThreadScrolls(0)
     , m_cumulativeNumLayersDrawn(0)
     , m_cumulativeNumMissingTiles(0)
     , m_lastSentMemoryVisibleBytes(0)
     , m_lastSentMemoryVisibleAndNearbyBytes(0)
     , m_lastSentMemoryUseBytes(0)
     , m_animationRegistrar(AnimationRegistrar::create())
@@ skipping 840 matching lines @@
     m_activeTree->DidBecomeActive();
 
     // Reduce wasted memory now that unlinked resources are guaranteed not
     // to be used.
     m_client->reduceWastedContentsTextureMemoryOnImplThread();
 
     m_client->onCanDrawStateChanged(canDraw());
     m_client->onHasPendingTreeStateChanged(pendingTree());
     m_client->setNeedsRedrawOnImplThread();
     m_client->renewTreePriority();
+
+    if (m_tileManager && m_debugState.continuousPainting) {
+        DCHECK(m_paintTimeCounter.get());
+        RenderingStats stats;
+        m_tileManager->GetRenderingStats(&stats);
+        m_paintTimeCounter->SaveRasterizeTime(stats.totalRasterizeTimeForNowBinsOnPendingTree, m_activeTree->source_frame_number());
+    }
 }
 
 void LayerTreeHostImpl::setVisible(bool visible)
 {
     DCHECK(m_proxy->isImplThread());
 
     if (m_visible == visible)
         return;
     m_visible = visible;
     didVisibilityChange(this, m_visible);
@@ skipping 694 matching lines @@
 
 skia::RefPtr<SkPicture> LayerTreeHostImpl::capturePicture()
 {
     LayerTreeImpl* tree = pendingTree() ? pendingTree() : activeTree();
     LayerImpl* layer = getNonCompositedContentLayerRecursive(tree->RootLayer());
     return layer ? layer->getPicture() : skia::RefPtr<SkPicture>();
 }
 
 void LayerTreeHostImpl::setDebugState(const LayerTreeDebugState& debugState)
 {
+    if (m_debugState.continuousPainting != debugState.continuousPainting) {
+        if (!m_paintTimeCounter.get())
+            m_paintTimeCounter = PaintTimeCounter::create();
+        m_paintTimeCounter->ClearHistory();
+    }
+
     m_debugState = debugState;
 
     if (m_tileManager)
         m_tileManager->SetRecordRenderingStats(m_debugState.recordRenderingStats());
 }
 
-void LayerTreeHostImpl::savePaintTime(const base::TimeDelta& totalPaintTime)
+void LayerTreeHostImpl::savePaintTime(const base::TimeDelta& totalPaintTime, const int& frameNumber)
 {
-    m_paintTimeCounter->SavePaintTime(totalPaintTime);
+    DCHECK(m_debugState.continuousPainting);
+    DCHECK(m_paintTimeCounter.get());
+    m_paintTimeCounter->SavePaintTime(totalPaintTime, frameNumber);

[nduca, 2013/02/13 08:12:21]

maybe we should use the word commitNumber here and throughout? It looks like its
a typo that we say source_frame_number on the impl side elsewhere

 }
 
 }  // namespace cc