cc: Do GatherPixelRefs from skia at record time

cc/resources/tile_manager.cc

Index: cc/resources/tile_manager.cc
diff --git a/cc/resources/tile_manager.cc b/cc/resources/tile_manager.cc
index b93cd69710a93ffdaf5f51a93a90b75953f6fa7c..b2d6a1de7f256d63cc363ba0a8fbbb0b1281a0ad 100644
--- a/cc/resources/tile_manager.cc
+++ b/cc/resources/tile_manager.cc
@@ -641,10 +641,15 @@ void TileManager::GatherPixelRefsForTile(Tile* tile) {
   if (managed_tile_state.need_to_gather_pixel_refs) {
     base::TimeTicks start_time =
         rendering_stats_instrumentation_->StartRecording();
-    tile->picture_pile()->GatherPixelRefs(
-        tile->content_rect_,
-        tile->contents_scale_,
-        managed_tile_state.pending_pixel_refs);
+    for (PicturePileImpl::LazyPixelRefIterator pixel_ref_iter(

[enne (OOO), 2013/04/22 19:10:56]

Looks fine to me.  I think I generally indent 5 spaces for the other two
statements to match up with the parenthesis in "for (", but I think what you've
written is legal style.

If you're not already using it, I highly recommend clang-format.  It's not
always right, but it normally does a good job even with these sorts of
constructs.

+            tile->content_rect(),
+            tile->contents_scale(),
+            tile->picture_pile());
+        pixel_ref_iter;
+        ++pixel_ref_iter) {
+      managed_tile_state.pending_pixel_refs.push_back(*pixel_ref_iter);
+      ++pixel_ref_iter;
+    }
     managed_tile_state.need_to_gather_pixel_refs = false;
     base::TimeDelta duration =
         rendering_stats_instrumentation_->EndRecording(start_time);
@@ -814,7 +819,6 @@ void TileManager::OnRasterTaskCompleted(
 
   // Finish resource initialization if |can_use_gpu_memory| is true.
   if (managed_tile_state.can_use_gpu_memory) {
-
     // Tile resources can't be freed until upload has completed.
     tile->drawing_info().can_be_freed_ = false;