Don't spam the GPU process with memory usage stats

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 14 matching lines @@
 #include "cc/quad_culler.h"
 #include "cc/render_pass_draw_quad.h"
 #include "cc/rendering_stats.h"
 #include "cc/scrollbar_animation_controller.h"
 #include "cc/scrollbar_layer_impl.h"
 #include "cc/shared_quad_state.h"
 #include "cc/single_thread_proxy.h"
 #include "cc/software_renderer.h"
 #include "cc/solid_color_draw_quad.h"
 #include "cc/texture_uploader.h"
+#include "cc/util.h"
 #include "ui/gfx/size_conversions.h"
 #include "ui/gfx/vector2d_conversions.h"
 
 namespace {
 
 void didVisibilityChange(cc::LayerTreeHostImpl* id, bool visible)
 {
     if (visible) {
         TRACE_EVENT_ASYNC_BEGIN1("webkit", "LayerTreeHostImpl::setVisible", id, "LayerTreeHostImpl", id);
         return;
@@ skipping 175 matching lines @@
                             PriorityCalculator::allowNothingCutoff())
     , m_backgroundColor(0)
     , m_hasTransparentBackground(false)
     , m_needsAnimateLayers(false)
     , m_pinchGestureActive(false)
     , m_fpsCounter(FrameRateCounter::create(m_proxy->hasImplThread()))
     , m_debugRectHistory(DebugRectHistory::create())
     , m_numImplThreadScrolls(0)
     , m_numMainThreadScrolls(0)
     , m_cumulativeNumLayersDrawn(0)
+    , m_lastSentMemoryVisibleBytes(0)
+    , m_lastSentMemoryVisibleAndNearbyBytes(0)
+    , m_lastSentMemoryUseBytes(0)
 {
     DCHECK(m_proxy->isImplThread());
     didVisibilityChange(this, m_visible);
 }
 
 LayerTreeHostImpl::~LayerTreeHostImpl()
 {
     DCHECK(m_proxy->isImplThread());
     TRACE_EVENT0("cc", "LayerTreeHostImpl::~LayerTreeHostImpl()");
 
@@ skipping 1372 matching lines @@
     stats->numFramesSentToScreen = fpsCounter()->currentFrameNumber();
     stats->droppedFrameCount = fpsCounter()->droppedFrameCount();
     stats->numImplThreadScrolls = m_numImplThreadScrolls;
     stats->numMainThreadScrolls = m_numMainThreadScrolls;
     stats->numLayersDrawn = m_cumulativeNumLayersDrawn;
 
     if (m_tileManager)
         m_tileManager->renderingStats(stats);
 }
 
+void LayerTreeHostImpl::sendManagedMemoryStats(
+    size_t memoryVisibleBytes,
+    size_t memoryVisibleAndNearbyBytes,
+    size_t memoryUseBytes)
+{
+    if (!renderer())
+      return;
+
+    // Round the numbers being sent up to the next 8MB, to throttle the rate
+    // at which we spam the GPU process.
+    const size_t roundingStep = 8 * 1024 * 1024;

[jamesr, 2012/12/07 23:19:12]

nit: "static const ...", so you don't get another copy on the stack every time
you call this

[ccameron, 2012/12/08 00:21:15]

Done.

+    memoryVisibleBytes = RoundUp(memoryVisibleBytes, roundingStep);
+    memoryVisibleAndNearbyBytes = RoundUp(memoryVisibleAndNearbyBytes, roundingStep);
+    memoryUseBytes = RoundUp(memoryUseBytes, roundingStep);
+    if (m_lastSentMemoryVisibleBytes == memoryVisibleBytes &&
+        m_lastSentMemoryVisibleAndNearbyBytes == memoryVisibleAndNearbyBytes &&
+        m_lastSentMemoryUseBytes == memoryUseBytes) {
+        return;
+    }
+    m_lastSentMemoryVisibleBytes = memoryVisibleBytes;
+    m_lastSentMemoryVisibleAndNearbyBytes = memoryVisibleAndNearbyBytes;
+    m_lastSentMemoryUseBytes = memoryUseBytes;
+
+    renderer()->sendManagedMemoryStats(m_lastSentMemoryVisibleBytes,
+                                       m_lastSentMemoryVisibleAndNearbyBytes,
+                                       m_lastSentMemoryUseBytes);
+}
+
 void LayerTreeHostImpl::animateScrollbars(base::TimeTicks time)
 {
     animateScrollbarsRecursive(m_rootLayerImpl.get(), time);
 }
 
 void LayerTreeHostImpl::animateScrollbarsRecursive(LayerImpl* layer, base::TimeTicks time)
 {
     if (!layer)
         return;
 
     ScrollbarAnimationController* scrollbarController = layer->scrollbarAnimationController();
     double monotonicTime = (time - base::TimeTicks()).InSecondsF();
     if (scrollbarController && scrollbarController->animate(monotonicTime))
         m_client->setNeedsRedrawOnImplThread();
 
     for (size_t i = 0; i < layer->children().size(); ++i)
         animateScrollbarsRecursive(layer->children()[i], time);
 }
 
 }  // namespace cc