cc: Optimize decode scheduling for checker-images.

cc/tiles/tile.h

Index: cc/tiles/tile.h
diff --git a/cc/tiles/tile.h b/cc/tiles/tile.h
index 435d5f0552e2e3c289f1f1cc227d80443f5a0cff..da0d44d9ac4cd1a996932b90286839da26ee6929 100644
--- a/cc/tiles/tile.h
+++ b/cc/tiles/tile.h
@@ -10,6 +10,7 @@
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
+#include "cc/paint/draw_image.h"
 #include "cc/raster/tile_task.h"
 #include "cc/tiles/tile_draw_info.h"
 #include "ui/gfx/geometry/axis_transform2d.h"
@@ -117,6 +118,16 @@ class CC_EXPORT Tile {
     return is_solid_color_analysis_performed_;
   }
 
+  bool set_raster_task_scheduled_with_checker_images(bool has_checker_images) {
+    bool previous_raster_task_scheduled_with_checker_images =

[vmpstr, 2017/04/18 00:20:00]

just call it 'return_value' or maybe even just make this a void function, since
the caller can always call the next function to check the current state if
desired.

[Khushal, 2017/04/19 06:16:41]

Changed it to |previous_value|. The call-site was initially using the accessor
but I changed it to return the value from the setter after you suggested that in
a previous patchset. :P I guess it does look prettier this way. I'm good with
either, let me know if you have a preference.

+        raster_task_scheduled_with_checker_images_;
+    raster_task_scheduled_with_checker_images_ = has_checker_images;
+    return previous_raster_task_scheduled_with_checker_images;
+  }
+  bool raster_task_scheduled_with_checker_images() const {
+    return raster_task_scheduled_with_checker_images_;
+  }
+
   const PictureLayerTiling* tiling() const { return tiling_; }
   void set_tiling(const PictureLayerTiling* tiling) { tiling_ = tiling; }
 
@@ -158,6 +169,10 @@ class CC_EXPORT Tile {
   Id invalidated_id_;
 
   unsigned scheduled_priority_;
+
+  // Set to true if there is a raster task scheduled for this tile that will
+  // rasterize a resource with checker images.
+  bool raster_task_scheduled_with_checker_images_ = false;
   scoped_refptr<TileTask> raster_task_;
 
   DISALLOW_COPY_AND_ASSIGN(Tile);