cc: Consolidate the analysis_canvas operations

cc/tile.h

Index: cc/tile.h
diff --git a/cc/tile.h b/cc/tile.h
index 1c988c2dc998e95910cb4de1493e23b793876bd8..63dbff872c54753381e3b2c65305d04f0248be62 100644
--- a/cc/tile.h
+++ b/cc/tile.h
@@ -61,6 +61,41 @@ class CC_EXPORT Tile : public base::RefCounted<Tile> {
     return managed_state_.resource->id();
   }
 
+  bool is_ready_to_draw() const {

[reveman, 2013/02/28 18:54:55]

Lower case function names are usually only used for simple inlined accessors and
mutators. I think this should be renamed to IsReadyToDraw and impl moved to .cc
as you need to call GetResourceId().

+    return GetResourceId() != 0 ||
+           is_solid_color_ ||
+           is_transparent_;
+  }
+
+  bool is_solid_color() const {
+    return is_solid_color_;
+  }
+
+  void set_solid_color(const SkColor& color) {
+    is_solid_color_ = true;
+    solid_color_ = color;
+  }
+
+  bool is_transparent() const {
+    return is_transparent_;
+  }
+
+  void set_transparent(bool flag) {
+    is_transparent_ = flag;
+  }
+
+  SkColor get_solid_color() const {
+    return solid_color_;
+  }
+
+  void set_cheap(bool flag) {
+    is_cheap_ = flag;
+  }
+
+  bool is_cheap() const {
+    return is_cheap_;
+  }
+
   const gfx::Rect& opaque_rect() const { return opaque_rect_; }
 
   bool contents_swizzled() const { return managed_state_.contents_swizzled; }
@@ -105,6 +140,11 @@ class CC_EXPORT Tile : public base::RefCounted<Tile> {
   TilePriority priority_[NUM_BIN_PRIORITIES];
   ManagedTileState managed_state_;
   int layer_id_;
+
+  SkColor solid_color_;
+  bool is_solid_color_;
+  bool is_transparent_;
+  bool is_cheap_;
 };
 
 }  // namespace cc