SetNeedsRedraw directly when updating a visible tile.

cc/resources/tile_manager.cc

Index: cc/resources/tile_manager.cc
diff --git a/cc/resources/tile_manager.cc b/cc/resources/tile_manager.cc
index 77c4df607639fd51e57cb22c35d6c344d618e200..d510e1038147b418501e2749daaa0369f03370df 100644
--- a/cc/resources/tile_manager.cc
+++ b/cc/resources/tile_manager.cc
@@ -372,7 +372,9 @@ void TileManager::DidFinishRunningTasks(TaskSet task_set) {
         }
 
         mts.draw_info.set_rasterize_on_demand();
-        client_->NotifyTileStateChanged(tile);
+        bool visible_tile =
+            tile->priority(ACTIVE_TREE).distance_to_visible == 0.f;
+        client_->NotifyTileStateChanged(tile, visible_tile);
       }
       raster_priority_queue_.Pop();
     }
@@ -429,6 +431,7 @@ void TileManager::ManageTiles(const GlobalStateThatImpactsTilePriority& state) {
 }
 
 bool TileManager::UpdateVisibleTiles() {
+  return false;
   TRACE_EVENT0("cc", "TileManager::UpdateVisibleTiles");
 
   rasterizer_->CheckForCompletedTasks();

[vmpstr, 2014/10/28 01:35:34]

We still need something to check for completed tasks before drawing, otherwise
we won't actually issue completion callbacks (and not initialize tiles). Maybe?

[enne (OOO), 2014/10/28 18:09:21]

Yeah.  If we're using async uploads, we need to check for completed tasks or no
tile will be ready even if it is.

[danakj, 2014/10/31 20:02:01]

Right, no longer changing this method.

@@ -656,8 +659,10 @@ void TileManager::FreeResourcesForTileAndNotifyClientIfTileWasReadyToDraw(
     Tile* tile) {
   bool was_ready_to_draw = tile->IsReadyToDraw();
   FreeResourcesForTile(tile);
-  if (was_ready_to_draw)
-    client_->NotifyTileStateChanged(tile);
+  if (was_ready_to_draw) {
+    bool visible_tile = tile->priority(ACTIVE_TREE).distance_to_visible == 0.f;
+    client_->NotifyTileStateChanged(tile, visible_tile);
+  }
 }
 
 void TileManager::ScheduleTasks(
@@ -821,10 +826,13 @@ void TileManager::OnRasterTaskCompleted(
     mts.draw_info.resource_ = resource.Pass();
   }
 
-  if (tile->priority(ACTIVE_TREE).distance_to_visible == 0.f)
+  bool visible_tile = false;
+  if (tile->priority(ACTIVE_TREE).distance_to_visible == 0.f) {
+    visible_tile = true;
     did_initialize_visible_tile_ = true;
+  }
 
-  client_->NotifyTileStateChanged(tile);
+  client_->NotifyTileStateChanged(tile, visible_tile);
 }
 
 scoped_refptr<Tile> TileManager::CreateTile(PicturePileImpl* picture_pile,