CC: Add a scheduled action for ManageTiles

cc/resources/tile_manager.cc

 // 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.
 
 #include "cc/resources/tile_manager.h"
 
 #include <algorithm>
 #include <limits>
 #include <string>
 
@@ skipping 223 matching lines @@
   AssignGpuMemoryToTiles(GetPrioritizedTileSet(),
                          &tiles_that_need_to_be_rasterized);
 
   // |tiles_that_need_to_be_rasterized| will be empty when we reach a
   // steady memory state. Keep scheduling tasks until we reach this state.
   if (!tiles_that_need_to_be_rasterized.empty()) {
     ScheduleTasks(tiles_that_need_to_be_rasterized);
     return;
   }
 
+  // We don't reserve memory for required-for-activate tiles during smoothness
+  // mode, so it is very likely these tiles will not get memory. So we prevent
+  // raster-on-demand and simply don't activate in this case.
+  bool allow_rasterize_on_demand = global_state_.tree_priority
+                                   != SMOOTHNESS_TAKES_PRIORITY;
+  bool can_activate = true;

[reveman, 2013/09/03 19:03:21]

can you drop this and just early out below when you find a tile that need
raster-on-demand but "global_state_.tree_priority == SMOOTHNESS_TAKES_PRIORITY"?

[epenner, 2013/09/03 19:36:29]

Good one! Done.

+
   // Use on-demand raster for any required-for-activation tiles that have not
   // been been assigned memory after reaching a steady memory state. This
   // ensures that we activate even when OOM.
   for (TileMap::iterator it = tiles_.begin(); it != tiles_.end(); ++it) {
     Tile* tile = it->second;
     ManagedTileState& mts = tile->managed_state();
     ManagedTileState::TileVersion& tile_version =
         mts.tile_versions[mts.raster_mode];
 
-    if (tile->required_for_activation() && !tile_version.IsReadyToDraw())
-      tile_version.set_rasterize_on_demand();
+    if (tile->required_for_activation() && !tile_version.IsReadyToDraw()) {
+      if (allow_rasterize_on_demand)
+        tile_version.set_rasterize_on_demand();
+      else
+        can_activate = false;
+    }
   }
 
-  client_->NotifyReadyToActivate();
+  if (can_activate)
+    client_->NotifyReadyToActivate();
 }
 
 void TileManager::DidFinishRunningTasksRequiredForActivation() {
   // This is only a true indication that all tiles required for
   // activation are initialized when no tiles are OOM. We need to
   // wait for DidFinishRunningTasks() to be called, try to re-assign
   // memory and in worst case use on-demand raster when tiles
   // required for activation are OOM.
   if (!all_tiles_required_for_activation_have_memory_)
     return;
@@ skipping 580 matching lines @@
     bytes_releasable_ += tile->bytes_consumed_if_allocated();
     ++resources_releasable_;
   }
 
   FreeUnusedResourcesForTile(tile);
   if (tile->priority(ACTIVE_TREE).distance_to_visible_in_pixels == 0)
     did_initialize_visible_tile_ = true;
 }
 
 }  // namespace cc