cc: Optimize decode scheduling for checker-images.

cc/tiles/tile.h

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CC_TILES_TILE_H_
 #define CC_TILES_TILE_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
 #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"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace cc {
 
 class PictureLayerTiling;
 class TileManager;
@@ skipping 87 matching lines @@
 
   bool HasRasterTask() const { return !!raster_task_.get(); }
 
   void set_solid_color_analysis_performed(bool performed) {
     is_solid_color_analysis_performed_ = performed;
   }
   bool is_solid_color_analysis_performed() const {
     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; }
 
  private:
   friend class TileManager;
   friend class FakeTileManager;
   friend class FakePictureLayerImpl;
 
   // Methods called by by tile manager.
   Tile(TileManager* tile_manager,
@@ skipping 21 matching lines @@
 
   Id id_;
 
   // The rect bounding the changes in this Tile vs the previous tile it
   // replaced.
   gfx::Rect invalidated_content_rect_;
   // The |id_| of the Tile that was invalidated and replaced by this 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);
 };
 
 }  // namespace cc
 
 #endif  // CC_TILES_TILE_H_