[cc] Pass metadata about a tile to the raster system for benchmarks

cc/tile_manager.cc

Index: cc/tile_manager.cc
diff --git a/cc/tile_manager.cc b/cc/tile_manager.cc
index f4c8bddbd93956b6214476603715d8039b1a0327..01d3cc696d65007f38704c1a505e6b41b2727e7d 100644
--- a/cc/tile_manager.cc
+++ b/cc/tile_manager.cc
@@ -759,6 +759,18 @@ scoped_ptr<ResourcePool::Resource> TileManager::PrepareTileForRaster(
   return resource.Pass();
 }
 
+namespace {

[reveman, 2013/02/13 10:24:39]

we already have an anonymous namespace section at the top, could you put this
there instead?

+
+RasterTaskMetadata GetRasterTaskMetadata(const ManagedTileState& mts) {
+  RasterTaskMetadata raster_task_metadata;
+  raster_task_metadata.is_tile_in_pending_tree_now_bin =
+      mts.tree_bin[PENDING_TREE] == NOW_BIN;
+  raster_task_metadata.tile_resolution = mts.resolution;
+  return raster_task_metadata;
+}
+
+} // namespace
+
 void TileManager::DispatchOneRasterTask(scoped_refptr<Tile> tile) {
   TRACE_EVENT0("cc", "TileManager::DispatchOneRasterTask");
   scoped_ptr<ResourcePool::Resource> resource = PrepareTileForRaster(tile);
@@ -771,7 +783,8 @@ void TileManager::DispatchOneRasterTask(scoped_refptr<Tile> tile) {
                      resource_id),
                  tile->content_rect_,
                  tile->contents_scale(),
-                 use_cheapness_estimator_),
+                 use_cheapness_estimator_,
+                 GetRasterTaskMetadata(tile->managed_state())),
       base::Bind(&TileManager::OnRasterTaskCompleted,
                  base::Unretained(this),
                  tile,
@@ -788,6 +801,7 @@ void TileManager::PerformOneRaster(Tile* tile) {
                 tile->content_rect_,
                 tile->contents_scale(),
                 use_cheapness_estimator_,
+                GetRasterTaskMetadata(tile->managed_state()),
                 tile->picture_pile(),
                 &rendering_stats_);
 
@@ -893,9 +907,12 @@ void TileManager::PerformRaster(uint8* buffer,
                                 const gfx::Rect& rect,
                                 float contents_scale,
                                 bool use_cheapness_estimator,
+                                const RasterTaskMetadata& raster_task_metadata,
                                 PicturePileImpl* picture_pile,
                                 RenderingStats* stats) {
-  TRACE_EVENT0("cc", "TileManager::PerformRaster");
+  TRACE_EVENT2("cc", "TileManager::PerformRaster",
+               "is_on_pending_tree", raster_task_metadata.is_tile_in_pending_tree_now_bin,
+               "is_low_res", raster_task_metadata.tile_resolution == LOW_RESOLUTION);

[reveman, 2013/02/13 10:24:39]

nit: break these lines so they are not too long.

   DCHECK(picture_pile);
   DCHECK(buffer);
   SkBitmap bitmap;