Add picture pile raster times histogram

cc/tile_manager.cc

 // Copyright 2012 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/tile_manager.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "base/json/json_writer.h"
 #include "base/logging.h"
+#include "base/metrics/histogram.h"
 #include "cc/platform_color.h"
 #include "cc/raster_worker_pool.h"
 #include "cc/resource_pool.h"
 #include "cc/tile.h"
 #include "third_party/skia/include/core/SkDevice.h"
 
 namespace cc {
 
 namespace {
 
@@ skipping 732 matching lines @@
                                 PicturePileImpl* picture_pile,
                                 RenderingStats* stats) {
   TRACE_EVENT0("cc", "TileManager::RunRasterTask");
   DCHECK(picture_pile);
   DCHECK(buffer);
   SkBitmap bitmap;
   bitmap.setConfig(SkBitmap::kARGB_8888_Config, rect.width(), rect.height());
   bitmap.setPixels(buffer);
   SkDevice device(bitmap);
   SkCanvas canvas(&device);
+
+  base::TimeTicks rasterize_begin_time;
+  if (stats)
+    rasterize_begin_time = base::TimeTicks::HighResNow();

[danakj, 2013/02/04 18:33:19]

This changed from Now to HighResNow? The ImageDecode doesn't use this. Was this
intended?

+
   picture_pile->Raster(&canvas, rect, contents_scale, stats);
+
+  if (stats) {
+    base::TimeTicks rasterize_end_time = base::TimeTicks::HighResNow();
+    base::TimeDelta rasterize_duration = rasterize_end_time - rasterize_begin_time;

[danakj, 2013/02/04 18:33:19]

80 col. if you drop the "rasterize_" prefix on these you won't have that
problemm and I think it will be just as clear in this context.

+    stats->totalRasterizeTime += rasterize_duration;
+    UMA_HISTOGRAM_CUSTOM_COUNTS(
+      "Renderer4.PictureRasterTimeMS",
+      rasterize_duration.InMilliseconds(),
+      0, 10, 10
+    );
+  }
 }
 
 // static
 void TileManager::RunImageDecodeTask(skia::LazyPixelRef* pixel_ref,
                                      RenderingStats* stats) {
   TRACE_EVENT0("cc", "TileManager::RunImageDecodeTask");
   base::TimeTicks decode_begin_time;
   if (stats)
     decode_begin_time = base::TimeTicks::Now();
   pixel_ref->Decode();
   if (stats) {
     stats->totalDeferredImageDecodeCount++;
     stats->totalDeferredImageDecodeTime +=
         base::TimeTicks::Now() - decode_begin_time;
   }
 }
 
 }  // namespace cc