Add picture pile raster times histogram

cc/tile_manager.cc

Index: cc/tile_manager.cc
diff --git a/cc/tile_manager.cc b/cc/tile_manager.cc
index 9935de5d2e84e1a130b685ac8ef0864213a46ae0..3711e37ca1597262cf828f8a5d17b11ad6e9a076 100644
--- a/cc/tile_manager.cc
+++ b/cc/tile_manager.cc
@@ -10,6 +10,7 @@
 #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"
@@ -762,7 +763,23 @@ void TileManager::RunRasterTask(uint8* buffer,
   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