cc: Combine analysis and raster

cc/test/fake_picture_pile_impl.cc

Index: cc/test/fake_picture_pile_impl.cc
diff --git a/cc/test/fake_picture_pile_impl.cc b/cc/test/fake_picture_pile_impl.cc
index 989c4f9f7a2bb94bed2976ec20a72103a7cbf1b0..b5bc37a095230bb9f4acdeec3904c29e6bc01aff 100644
--- a/cc/test/fake_picture_pile_impl.cc
+++ b/cc/test/fake_picture_pile_impl.cc
@@ -94,4 +94,26 @@ void FakePicturePileImpl::RerecordPile() {
   }
 }
 
+void FakePicturePileImpl::AnalyzeInRect(gfx::Rect content_rect,
+                                        float contents_scale,
+                                        Analysis* analysis) {
+  // Create and raster to a bitmap of content_rect size, even though the
+  // analysis_rect may be smaller to simulate edge tiles where recorded content
+  // doesn't cover the entire content_rect.
+  SkBitmap empty_bitmap;
+  empty_bitmap.setConfig(SkBitmap::kNo_Config,

[Stephen White, 2013/11/07 18:14:22]

Nit: I find it a bit weird that this API uses kNo_Config to indicate a new
bitmap, but I see that that usage predates this CL. I wonder if there might be a
cleaner way, perhaps using getTexture() for GPU, or getPixels() for raster.

[enne (OOO), 2013/11/07 22:36:10]

If there's a better way to fix this, please let me know or uploa da patch.  I am
doing this because other code did this and not because I know what I should be
doing with Skia.

+                         content_rect.width(),
+                         content_rect.height());
+
+  gfx::Rect analysis_rect(
+      AnalysisRectForRaster(content_rect, contents_scale));
+  skia::AnalysisDevice device(empty_bitmap, analysis_rect);
+  skia::AnalysisCanvas canvas(&device);
+
+  RasterDirect(&canvas, content_rect, contents_scale, NULL);
+
+  analysis->is_solid_color = canvas.GetColorIfSolid(&analysis->solid_color);
+  analysis->has_text = canvas.HasText();
+}
+
 }  // namespace cc