Expose IsSolidColor and write units tests

cc/resources/picture.h

Index: cc/resources/picture.h
diff --git a/cc/resources/picture.h b/cc/resources/picture.h
index 78833d17e9144c0721ec888d216af7e94577d2cb..db853aeb07c5a18141e9992e59b0b3118c6a84f7 100644
--- a/cc/resources/picture.h
+++ b/cc/resources/picture.h
@@ -74,6 +74,7 @@ class CC_EXPORT Picture
   bool HasRecording() const { return picture_.get() != NULL; }
 
   bool IsSuitableForGpuRasterization() const;
+  int GetApproximateOpCount() const;

[enne (OOO), 2014/08/20 18:19:26]

style nit: No need for "get" in a function name.

[hendrikw, 2014/08/20 20:46:55]

vmpstr suggested the name :)  I think Get improves readabilty (and in some cases
reduces ambiguity), but I've removed it anyway, in this case it doesn't matter

 
   // Apply this scale and raster the negated region into the canvas.
   // |negated_content_region| specifies the region to be clipped out of the
@@ -84,6 +85,8 @@ class CC_EXPORT Picture
              const Region& negated_content_region,
              float contents_scale);
 
+  void RasterForAnalysis(SkCanvas* canvas);
+
   // Draw the picture directly into the given canvas, without applying any
   // clip/scale/layer transformations.
   void Replay(SkCanvas* canvas);