cc: Switch eviction iterators to consider combined priority.

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 512 matching lines @@
     MemoryUsage* usage) {
   while (usage->Exceeds(limit)) {
     if (!eviction_priority_queue) {
       eviction_priority_queue =
           client_->BuildEvictionQueue(global_state_.tree_priority);
     }
     if (eviction_priority_queue->IsEmpty())
       break;
 
     Tile* tile = eviction_priority_queue->Top();
-    if (!other_priority.IsHigherPriorityThan(tile->combined_priority()))
+    const TilePriority& current_priority = tile->combined_priority();
+    if (!tile->is_occluded_combined() &&
+        (current_priority.resolution != NON_IDEAL_RESOLUTION) &&

[vmiura, 2015/02/12 22:18:41]

What are the implications of these additional checks?

[vmpstr, 2015/02/12 23:38:19]

This is just making us be able to have more memory for rasterization. Without
this a tile of a non ideal scale can block rasterization of a high res tile for
instance, because we don't have memory... This is kind of an optional change,
but I think it makes it a bit more robust. Let me know if you prefer I did this
separately (or didn't do it at all :) ) (I also changed the condition a bit and
added comments).

+        !other_priority.IsHigherPriorityThan(tile->combined_priority())) {
       break;
+    }
 
     *usage -= MemoryUsage::FromTile(tile);
     FreeResourcesForTileAndNotifyClientIfTileWasReadyToDraw(tile);
     eviction_priority_queue->Pop();
   }
   return eviction_priority_queue;
 }
 
 bool TileManager::TilePriorityViolatesMemoryPolicy(
     const TilePriority& priority) {
@@ skipping 504 matching lines @@
   result -= other;
   return result;
 }
 
 bool TileManager::MemoryUsage::Exceeds(const MemoryUsage& limit) const {
   return memory_bytes_ > limit.memory_bytes_ ||
          resource_count_ > limit.resource_count_;
 }
 
 }  // namespace cc