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 16d37d1491d88a5f9b7938eada47ed21ea32d2d2..37980a1a3317cd390005412bc80b83a4b0b25e01 100644
--- a/cc/tiles/tile.h
+++ b/cc/tiles/tile.h
@@ -113,6 +113,10 @@ class CC_EXPORT Tile {
     return is_solid_color_analysis_performed_;
   }
 
+  bool raster_task_scheduled_with_checker_images() const {

[vmpstr, 2017/03/29 19:11:45]

Add a setter as well. I want to remove the friend TileManager on the tile
eventually.

This needs a comment: unlike "solid color analysis performed" which mean we've
done the analysis, this could be interpreted that the raster task is _currently_
scheduled, vs raster task has _ever_ been scheduled. Also, remove the
"raster_task" part here.

This also gives me an interesting thought: if we checker an image, can it be the
case that the tile afterwards is solid color? I'm not sure it's worth looking at
this, but something to consider in the future if we want to save some memory
while we're decoding the image.

[Khushal, 2017/03/29 22:10:20]

Its definitely going to be possible and something to optimize.

[Khushal, 2017/03/31 04:31:00]

Done.

+    return raster_task_scheduled_with_checker_images_;
+  }
+
   const PictureLayerTiling* tiling() const { return tiling_; }
   void set_tiling(const PictureLayerTiling* tiling) { tiling_ = tiling; }
 
@@ -154,6 +158,7 @@ class CC_EXPORT Tile {
   Id invalidated_id_;
 
   unsigned scheduled_priority_;
+  bool raster_task_scheduled_with_checker_images_ = false;
   scoped_refptr<TileTask> raster_task_;
 
   DISALLOW_COPY_AND_ASSIGN(Tile);