SetNeedsRedraw directly when updating a visible tile.

cc/resources/tile_manager.h

Index: cc/resources/tile_manager.h
diff --git a/cc/resources/tile_manager.h b/cc/resources/tile_manager.h
index f5bea383141a38c325f695ed2247e9b7561331a9..7812818f287b26e84bf09f2883ed1c14cad4706a 100644
--- a/cc/resources/tile_manager.h
+++ b/cc/resources/tile_manager.h
@@ -104,8 +104,11 @@ class CC_EXPORT TileManager : public RasterizerClient,
 
   void ManageTiles(const GlobalStateThatImpactsTilePriority& state);
 
-  // Returns true when visible tiles have been initialized.
-  bool UpdateVisibleTiles();
+  void UpdateVisibleTiles();
+
+  bool is_top_priority_uninitialized_tile_visible_and_active() {
+    return is_top_priority_uninitialized_tile_visible_and_active_;
+  }

[reveman, 2014/10/31 00:44:01]

Still doesn't feel right to be reaching into the tile manager for this. Seems
backwards that LTHI should be asking the tile manager for this when LTHI is the
one providing the tile manager with a prioritized raster queue. It's like "hey
tile manager, I'm your client, here's the raster queue you're asking for", then
some time later, "oh, tile manager? you know that raster queue I gave you
earlier, did the top priority tile happen to be visible and on active tree by
any chance?"

Maybe some refactor can make this cleaner in the future but can you add a
LTHI::front_tile_in_last_built_raster_queue_was_visible_and_on_active_tree_
member that you set in BuildRasterQueue and use it in LTHI instead of this tile
manager function?

[danakj, 2014/10/31 15:39:46]

Ok thanks for bearing with me, I'm pretty clueless about TileManager. I managed
to move this to LTHI by adding a client callback when DidFinishRunningTasks
early-outs because it is done everything.

 
   scoped_refptr<Tile> CreateTile(RasterSource* raster_source,
                                  const gfx::Size& tile_size,
@@ -248,9 +251,9 @@ class CC_EXPORT TileManager : public RasterizerClient,
 
   RenderingStatsInstrumentation* rendering_stats_instrumentation_;
 
-  bool did_initialize_visible_tile_;
   bool did_check_for_completed_tasks_since_last_schedule_tasks_;
   bool did_oom_on_last_assign_;
+  bool is_top_priority_uninitialized_tile_visible_and_active_;
 
   typedef base::hash_map<uint32_t, scoped_refptr<ImageDecodeTask>>
       PixelRefTaskMap;