cc: Rework tiling set eviction iterator.

cc/resources/tiling_set_eviction_queue.cc

Index: cc/resources/tiling_set_eviction_queue.cc
diff --git a/cc/resources/tiling_set_eviction_queue.cc b/cc/resources/tiling_set_eviction_queue.cc
index 5ee678b53909962fba63e8f52ab275f460753bbe..43d085b629f7047e09ce1d8b22748115b0df8ccf 100644
--- a/cc/resources/tiling_set_eviction_queue.cc
+++ b/cc/resources/tiling_set_eviction_queue.cc
@@ -38,285 +38,432 @@ bool IsSharedOutOfOrderTile(WhichTree tree, const Tile* tile) {
 TilingSetEvictionQueue::TilingSetEvictionQueue(
     PictureLayerTilingSet* tiling_set,
     bool skip_shared_out_of_order_tiles)
-    : tiling_set_(tiling_set),
-      tree_(tiling_set->client()->GetTree()),
+    : tree_(tiling_set->client()->GetTree()),
       skip_shared_out_of_order_tiles_(skip_shared_out_of_order_tiles),
-      processing_soon_border_rect_(false),
-      processing_required_for_activation_tiles_(false),
-      current_priority_bin_(TilePriority::EVENTUALLY),
-      current_tiling_index_(0u),
-      current_tiling_range_type_(PictureLayerTilingSet::HIGHER_THAN_HIGH_RES),
-      current_eviction_tile_(nullptr) {
+      phase_(EVENTUALLY_RECT),
+      current_tile_(nullptr) {
   // Early out if the layer has no tilings.
-  if (!tiling_set_->num_tilings())
+  if (!tiling_set->num_tilings())
     return;
-
-  current_tiling_index_ = CurrentTilingRange().start - 1u;
-  AdvanceToNextValidTiling();
+  GenerateTilingOrder(tiling_set);
+  eventually_iterator_ = EventuallyTilingIterator(
+      &tilings_, tree_, skip_shared_out_of_order_tiles_);
+  if (eventually_iterator_.done()) {
+    AdvancePhase();
+    return;
+  }
+  current_tile_ = *eventually_iterator_;
 }
 
 TilingSetEvictionQueue::~TilingSetEvictionQueue() {
 }
 
-bool TilingSetEvictionQueue::IsEmpty() const {
-  return !current_eviction_tile_;
+void TilingSetEvictionQueue::GenerateTilingOrder(
+    PictureLayerTilingSet* tiling_set) {
+  tilings_.reserve(tiling_set->num_tilings());
+  // Generate all of the tilings in the order described in the header comment
+  // for this class.
+  auto range =
+      tiling_set->GetTilingRange(PictureLayerTilingSet::HIGHER_THAN_HIGH_RES);
+  for (int i = range.start; i < range.end; ++i)
+    tilings_.push_back(tiling_set->tiling_at(i));
+
+  range = tiling_set->GetTilingRange(PictureLayerTilingSet::LOWER_THAN_LOW_RES);
+  for (int i = range.end - 1; i >= range.start; --i)
+    tilings_.push_back(tiling_set->tiling_at(i));
+
+  range = tiling_set->GetTilingRange(
+      PictureLayerTilingSet::BETWEEN_HIGH_AND_LOW_RES);
+  for (int i = range.end - 1; i >= range.start; --i)
+    tilings_.push_back(tiling_set->tiling_at(i));
+
+  range = tiling_set->GetTilingRange(PictureLayerTilingSet::LOW_RES);
+  for (int i = range.start; i < range.end; ++i)
+    tilings_.push_back(tiling_set->tiling_at(i));
+
+  range = tiling_set->GetTilingRange(PictureLayerTilingSet::HIGH_RES);
+  for (int i = range.start; i < range.end; ++i)
+    tilings_.push_back(tiling_set->tiling_at(i));
+  DCHECK_EQ(tiling_set->num_tilings(), tilings_.size());
 }
 
-void TilingSetEvictionQueue::Pop() {
-  DCHECK(!IsEmpty());
+void TilingSetEvictionQueue::AdvancePhase() {
+  current_tile_ = nullptr;
+  while (!current_tile_ &&
+         phase_ != VISIBLE_RECT_REQUIRED_FOR_ACTIVATION_UNOCCLUDED) {
+    phase_ = static_cast<Phase>(phase_ + 1);
+    switch (phase_) {
+      case EVENTUALLY_RECT:
+        NOTREACHED();
+        break;
+      case SOON_BORDER_RECT:
+        soon_iterator_ = SoonBorderTilingIterator(
+            &tilings_, tree_, skip_shared_out_of_order_tiles_);
+        if (!soon_iterator_.done())
+          current_tile_ = *soon_iterator_;
+        break;
+      case SKEWPORT_RECT:
+        skewport_iterator_ = SkewportTilingIterator(
+            &tilings_, tree_, skip_shared_out_of_order_tiles_);
+        if (!skewport_iterator_.done())
+          current_tile_ = *skewport_iterator_;
+        break;
+      case VISIBLE_RECT_OCCLUDED:
+        visible_iterator_ = VisibleTilingIterator(
+            &tilings_, tree_, skip_shared_out_of_order_tiles_,
+            true /* return occluded tiles */,
+            false /* return required for activation tiles */);
+        if (!visible_iterator_.done())
+          current_tile_ = *visible_iterator_;
+        break;
+      case VISIBLE_RECT_UNOCCLUDED:
+        visible_iterator_ = VisibleTilingIterator(
+            &tilings_, tree_, skip_shared_out_of_order_tiles_,
+            false /* return occluded tiles */,
+            false /* return required for activation tiles */);
+        if (!visible_iterator_.done())
+          current_tile_ = *visible_iterator_;
+        break;
+      case VISIBLE_RECT_REQUIRED_FOR_ACTIVATION_OCCLUDED:
+        visible_iterator_ = VisibleTilingIterator(
+            &tilings_, tree_, skip_shared_out_of_order_tiles_,
+            true /* return occluded tiles */,
+            true /* return required for activation tiles */);
+        if (!visible_iterator_.done())
+          current_tile_ = *visible_iterator_;
+        break;
+      case VISIBLE_RECT_REQUIRED_FOR_ACTIVATION_UNOCCLUDED:
+        visible_iterator_ = VisibleTilingIterator(
+            &tilings_, tree_, skip_shared_out_of_order_tiles_,
+            false /* return occluded tiles */,
+            true /* return required for activation tiles */);
+        if (!visible_iterator_.done())
+          current_tile_ = *visible_iterator_;
+        break;
+    }
+  }
+}
 
-  if (!AdvanceToNextEvictionTile())
-    AdvanceToNextValidTiling();
+bool TilingSetEvictionQueue::IsEmpty() const {
+  return !current_tile_;
 }
 
 Tile* TilingSetEvictionQueue::Top() {
   DCHECK(!IsEmpty());
-  return current_eviction_tile_;
+  return current_tile_;
 }
 
 const Tile* TilingSetEvictionQueue::Top() const {
   DCHECK(!IsEmpty());
-  return current_eviction_tile_;
+  return current_tile_;
 }
 
-bool TilingSetEvictionQueue::AdvanceToNextEvictionTile() {
-  // Advance to the next eviction tile within the current priority bin and
-  // tiling. This is done while advancing to a new tiling and while popping
-  // the current tile.
-
-  bool required_for_activation = processing_required_for_activation_tiles_;
-
-  for (;;) {
-    while (spiral_iterator_) {
-      std::pair<int, int> next_index = spiral_iterator_.index();
-      Tile* tile = current_tiling_->TileAt(next_index.first, next_index.second);
-      ++spiral_iterator_;
-      if (!tile || !tile->HasResource())
-        continue;
-      current_tiling_->UpdateTileAndTwinPriority(tile);
-      if (skip_shared_out_of_order_tiles_ &&
-          IsSharedOutOfOrderTile(tree_, tile))
-        continue;
-      DCHECK_EQ(tile->required_for_activation(), required_for_activation);
-      current_eviction_tile_ = tile;
-      return true;
-    }
-    if (processing_soon_border_rect_) {
-      // Advance from soon border rect to skewport rect.
-      processing_soon_border_rect_ = false;
-      if (current_tiling_->has_skewport_rect_tiles_) {
-        spiral_iterator_ = TilingData::ReverseSpiralDifferenceIterator(
-            &current_tiling_->tiling_data_,
-            current_tiling_->current_skewport_rect_,
-            current_tiling_->current_visible_rect_,
-            current_tiling_->current_visible_rect_);
-        continue;
-      }
+void TilingSetEvictionQueue::Pop() {
+  DCHECK(!IsEmpty());
+  current_tile_ = nullptr;
+  switch (phase_) {
+    case EVENTUALLY_RECT:
+      ++eventually_iterator_;
+      if (!eventually_iterator_.done())
+        current_tile_ = *eventually_iterator_;
+      break;
+    case SOON_BORDER_RECT:
+      ++soon_iterator_;
+      if (!soon_iterator_.done())
+        current_tile_ = *soon_iterator_;
+      break;
+    case SKEWPORT_RECT:
+      ++skewport_iterator_;
+      if (!skewport_iterator_.done())
+        current_tile_ = *skewport_iterator_;
+      break;
+    case VISIBLE_RECT_OCCLUDED:
+    case VISIBLE_RECT_UNOCCLUDED:
+    case VISIBLE_RECT_REQUIRED_FOR_ACTIVATION_OCCLUDED:
+    case VISIBLE_RECT_REQUIRED_FOR_ACTIVATION_UNOCCLUDED:
+      ++visible_iterator_;
+      if (!visible_iterator_.done())
+        current_tile_ = *visible_iterator_;
+      break;
+  }
+  if (!current_tile_)
+    AdvancePhase();
+}
+
+// EvictionRectIterator
+TilingSetEvictionQueue::EvictionRectIterator::EvictionRectIterator()
+    : tile_(nullptr),
+      tilings_(nullptr),
+      tree_(ACTIVE_TREE),
+      skip_shared_out_of_order_tiles_(false),
+      tiling_index_(0) {
+}
+
+TilingSetEvictionQueue::EvictionRectIterator::EvictionRectIterator(
+    std::vector<PictureLayerTiling*>* tilings,
+    WhichTree tree,
+    bool skip_shared_out_of_order_tiles)
+    : tile_(nullptr),
+      tilings_(tilings),
+      tree_(tree),
+      skip_shared_out_of_order_tiles_(skip_shared_out_of_order_tiles),
+      tiling_index_(0) {
+}
+
+template <typename TilingIteratorType>
+bool TilingSetEvictionQueue::EvictionRectIterator::AdvanceToNextTile(
+    TilingIteratorType* iterator) {
+  bool found_tile = false;
+  while (!found_tile) {
+    ++(*iterator);
+    if (!(*iterator)) {
+      tile_ = nullptr;
+      break;
     }
-    break;
+    found_tile = GetFirstTileAndCheckIfValid(iterator);
   }
+  return found_tile;
+}
+
+template <typename TilingIteratorType>
+bool TilingSetEvictionQueue::EvictionRectIterator::GetFirstTileAndCheckIfValid(
+    TilingIteratorType* iterator) {
+  tile_ = (*tilings_)[tiling_index_]->TileAt(iterator->index_x(),
+                                             iterator->index_y());
+  // If there's nothing to evict, return false.
+  if (!tile_ || !tile_->HasResource())
+    return false;
+  (*tilings_)[tiling_index_]->UpdateTileAndTwinPriority(tile_);
+  // If the tile is out of order, return false.
+  if (skip_shared_out_of_order_tiles_ && IsSharedOutOfOrderTile(tree_, tile_))
+    return false;
+  // In other cases, the tile we got is a viable candidate, return true.
+  return true;
+}
 
-  TilePriority::PriorityBin max_tile_priority_bin =
-      current_tiling_->client_->GetMaxTilePriorityBin();
-  while (visible_iterator_) {
-    std::pair<int, int> next_index = visible_iterator_.index();
-    Tile* tile = current_tiling_->TileAt(next_index.first, next_index.second);
-    ++visible_iterator_;
-    if (!tile || !tile->HasResource())
+// EventuallyTilingIterator
+TilingSetEvictionQueue::EventuallyTilingIterator::EventuallyTilingIterator(
+    std::vector<PictureLayerTiling*>* tilings,
+    WhichTree tree,
+    bool skip_shared_out_of_order_tiles)
+    : EvictionRectIterator(tilings, tree, skip_shared_out_of_order_tiles) {
+  // Find the first tiling with a tile.
+  while (tiling_index_ < tilings_->size()) {
+    if (!((*tilings_))[tiling_index_]->has_eventually_rect_tiles()) {
+      ++tiling_index_;
       continue;
-    // If the max tile priority is not NOW, updated priorities for tiles
-    // returned by the visible iterator will not have NOW (but EVENTUALLY)
-    // priority bin and cannot therefore be required for activation tiles nor
-    // occluded NOW tiles in the current tiling.
-    if (max_tile_priority_bin <= TilePriority::NOW) {
-      // If the current tiling is a pending tree tiling, required for
-      // activation tiles can be detected without updating tile priorities.
-      if (tree_ == PENDING_TREE &&
-          current_tiling_->IsTileRequiredForActivationIfVisible(tile) !=
-              required_for_activation) {
-        continue;
-      }
-      // Unoccluded NOW tiles should be evicted (and thus returned) only after
-      // all occluded NOW tiles.
-      if (!current_tiling_->IsTileOccluded(tile)) {
-        unoccluded_now_tiles_.push_back(tile);
-        continue;
-      }
     }
-    current_tiling_->UpdateTileAndTwinPriority(tile);
-    if (skip_shared_out_of_order_tiles_ && IsSharedOutOfOrderTile(tree_, tile))
-      continue;
-    if (tile->required_for_activation() != required_for_activation)
+    iterator_ = TilingData::ReverseSpiralDifferenceIterator(
+        ((*tilings_))[tiling_index_]->tiling_data(),
+        ((*tilings_))[tiling_index_]->current_eventually_rect(),
+        ((*tilings_))[tiling_index_]->current_skewport_rect(),
+        ((*tilings_))[tiling_index_]->current_soon_border_rect());
+    if (!iterator_) {
+      ++tiling_index_;
       continue;
-    current_eviction_tile_ = tile;
-    return true;
+    }
+    break;
   }
+  if (tiling_index_ >= tilings_->size())
+    return;
+  if (!GetFirstTileAndCheckIfValid(&iterator_))
+    ++(*this);
+}
 
-  while (!unoccluded_now_tiles_.empty()) {
-    // All (unoccluded) NOW tiles have the same priority bin (NOW) and the same
-    // distance to visible (0.0), so it does not matter that tiles are popped
-    // in reversed (FILO) order.
-    Tile* tile = unoccluded_now_tiles_.back();
-    unoccluded_now_tiles_.pop_back();
-    DCHECK(tile);
-    if (!tile->HasResource())
-      continue;
-    current_tiling_->UpdateTileAndTwinPriority(tile);
-    if (skip_shared_out_of_order_tiles_ && IsSharedOutOfOrderTile(tree_, tile))
+TilingSetEvictionQueue::EventuallyTilingIterator&
+    TilingSetEvictionQueue::EventuallyTilingIterator::
+    operator++() {
+  bool found_tile = AdvanceToNextTile(&iterator_);
+  while (!found_tile && (tiling_index_ + 1) < tilings_->size()) {
+    ++tiling_index_;
+    if (!((*tilings_))[tiling_index_]->has_eventually_rect_tiles())
       continue;
-    if (tile->required_for_activation() != required_for_activation)
+    iterator_ = TilingData::ReverseSpiralDifferenceIterator(
+        ((*tilings_))[tiling_index_]->tiling_data(),
+        ((*tilings_))[tiling_index_]->current_eventually_rect(),
+        ((*tilings_))[tiling_index_]->current_skewport_rect(),
+        ((*tilings_))[tiling_index_]->current_soon_border_rect());
+    if (!iterator_)
       continue;
-    current_eviction_tile_ = tile;
-    return true;
+    found_tile = GetFirstTileAndCheckIfValid(&iterator_);
+    if (!found_tile)
+      found_tile = AdvanceToNextTile(&iterator_);
   }
-
-  current_eviction_tile_ = nullptr;
-  return false;
+  return *this;
 }
 
-bool TilingSetEvictionQueue::AdvanceToNextPriorityBin() {
-  // Advance to the next priority bin. This is done only after all tiling range
-  // types (including the required for activation tiling) within the previous
-  // priority bin have been gone through.
-  DCHECK_EQ(current_tiling_range_type_, PictureLayerTilingSet::HIGH_RES);
-
-  switch (current_priority_bin_) {
-    case TilePriority::EVENTUALLY:
-      current_priority_bin_ = TilePriority::SOON;
-      return true;
-    case TilePriority::SOON:
-      current_priority_bin_ = TilePriority::NOW;
-      return true;
-    case TilePriority::NOW:
-      return false;
+// SoonBorderTilingIterator
+TilingSetEvictionQueue::SoonBorderTilingIterator::SoonBorderTilingIterator(
+    std::vector<PictureLayerTiling*>* tilings,
+    WhichTree tree,
+    bool skip_shared_out_of_order_tiles)
+    : EvictionRectIterator(tilings, tree, skip_shared_out_of_order_tiles) {
+  // Find the first tiling with a tile.
+  while (tiling_index_ < tilings_->size()) {
+    if (!((*tilings_))[tiling_index_]->has_soon_border_rect_tiles()) {
+      ++tiling_index_;
+      continue;
+    }
+    iterator_ = TilingData::ReverseSpiralDifferenceIterator(
+        ((*tilings_))[tiling_index_]->tiling_data(),
+        ((*tilings_))[tiling_index_]->current_soon_border_rect(),
+        ((*tilings_))[tiling_index_]->current_skewport_rect(),
+        ((*tilings_))[tiling_index_]->current_visible_rect());
+    if (!iterator_) {
+      ++tiling_index_;
+      continue;
+    }
+    break;
   }
-  NOTREACHED();
-  return false;
+  if (tiling_index_ >= tilings_->size())
+    return;
+  if (!GetFirstTileAndCheckIfValid(&iterator_))
+    ++(*this);
 }
 
-bool TilingSetEvictionQueue::AdvanceToNextTilingRangeType() {
-  // Advance to the next tiling range type within the current priority bin, to
-  // the required for activation tiling range type within the current priority
-  // bin or to the first tiling range type within the next priority bin. This
-  // is done only after all tilings within the previous tiling range type have
-  // been gone through.
-  DCHECK_EQ(current_tiling_index_, CurrentTilingRange().end);
-
-  switch (current_tiling_range_type_) {
-    case PictureLayerTilingSet::HIGHER_THAN_HIGH_RES:
-      current_tiling_range_type_ = PictureLayerTilingSet::LOWER_THAN_LOW_RES;
-      return true;
-    case PictureLayerTilingSet::LOWER_THAN_LOW_RES:
-      current_tiling_range_type_ =
-          PictureLayerTilingSet::BETWEEN_HIGH_AND_LOW_RES;
-      return true;
-    case PictureLayerTilingSet::BETWEEN_HIGH_AND_LOW_RES:
-      current_tiling_range_type_ = PictureLayerTilingSet::LOW_RES;
-      return true;
-    case PictureLayerTilingSet::LOW_RES:
-      current_tiling_range_type_ = PictureLayerTilingSet::HIGH_RES;
-      return true;
-    case PictureLayerTilingSet::HIGH_RES:
-      // Process required for activation tiles (unless that has already been
-      // done). Only pending tree NOW tiles may be required for activation.
-      if (!processing_required_for_activation_tiles_ &&
-          current_priority_bin_ == TilePriority::NOW && tree_ == PENDING_TREE) {
-        processing_required_for_activation_tiles_ = true;
-        return true;
-      }
-      processing_required_for_activation_tiles_ = false;
-
-      if (!AdvanceToNextPriorityBin())
-        return false;
-
-      current_tiling_range_type_ = PictureLayerTilingSet::HIGHER_THAN_HIGH_RES;
-      return true;
+TilingSetEvictionQueue::SoonBorderTilingIterator&
+    TilingSetEvictionQueue::SoonBorderTilingIterator::
+    operator++() {
+  bool found_tile = AdvanceToNextTile(&iterator_);
+  while (!found_tile && (tiling_index_ + 1) < tilings_->size()) {
+    ++tiling_index_;
+    if (!((*tilings_))[tiling_index_]->has_soon_border_rect_tiles())
+      continue;
+    iterator_ = TilingData::ReverseSpiralDifferenceIterator(
+        ((*tilings_))[tiling_index_]->tiling_data(),
+        ((*tilings_))[tiling_index_]->current_soon_border_rect(),
+        ((*tilings_))[tiling_index_]->current_skewport_rect(),
+        ((*tilings_))[tiling_index_]->current_visible_rect());
+    if (!iterator_)
+      continue;
+    found_tile = GetFirstTileAndCheckIfValid(&iterator_);
+    if (!found_tile)
+      found_tile = AdvanceToNextTile(&iterator_);
   }
-  NOTREACHED();
-  return false;
+  return *this;
 }
 
-bool TilingSetEvictionQueue::AdvanceToNextValidTiling() {
-  // Advance to the next tiling within current tiling range type or to
-  // the first tiling within the next tiling range type or priority bin until
-  // the next eviction tile is found. This is done only after all eviction
-  // tiles within the previous tiling within the current priority bin and
-  // tiling range type have been gone through.
-  DCHECK(!current_eviction_tile_);
-  DCHECK_NE(current_tiling_index_, CurrentTilingRange().end);
-
-  for (;;) {
-    ++current_tiling_index_;
-    while (current_tiling_index_ == CurrentTilingRange().end) {
-      if (!AdvanceToNextTilingRangeType())
-        return false;
-      current_tiling_index_ = CurrentTilingRange().start;
+// SkewportTilingIterator
+TilingSetEvictionQueue::SkewportTilingIterator::SkewportTilingIterator(
+    std::vector<PictureLayerTiling*>* tilings,
+    WhichTree tree,
+    bool skip_shared_out_of_order_tiles)
+    : EvictionRectIterator(tilings, tree, skip_shared_out_of_order_tiles) {
+  // Find the first tiling with a tile.
+  while (tiling_index_ < tilings_->size()) {
+    if (!((*tilings_))[tiling_index_]->has_skewport_rect_tiles()) {
+      ++tiling_index_;
+      continue;
     }
-    current_tiling_ = tiling_set_->tiling_at(CurrentTilingIndex());
-
-    switch (current_priority_bin_) {
-      case TilePriority::EVENTUALLY:
-        if (current_tiling_->has_eventually_rect_tiles_) {
-          spiral_iterator_ = TilingData::ReverseSpiralDifferenceIterator(
-              &current_tiling_->tiling_data_,
-              current_tiling_->current_eventually_rect_,
-              current_tiling_->current_skewport_rect_,
-              current_tiling_->current_soon_border_rect_);
-          if (AdvanceToNextEvictionTile())
-            return true;
-        }
-        break;
-      case TilePriority::SOON:
-        if (current_tiling_->has_skewport_rect_tiles_ ||
-            current_tiling_->has_soon_border_rect_tiles_) {
-          processing_soon_border_rect_ = true;
-          if (current_tiling_->has_soon_border_rect_tiles_)
-            spiral_iterator_ = TilingData::ReverseSpiralDifferenceIterator(
-                &current_tiling_->tiling_data_,
-                current_tiling_->current_soon_border_rect_,
-                current_tiling_->current_skewport_rect_,
-                current_tiling_->current_visible_rect_);
-          if (AdvanceToNextEvictionTile())
-            return true;
-        }
-        break;
-      case TilePriority::NOW:
-        if (current_tiling_->has_visible_rect_tiles_) {
-          visible_iterator_ =
-              TilingData::Iterator(&current_tiling_->tiling_data_,
-                                   current_tiling_->current_visible_rect_,
-                                   false /* include_borders */);
-          if (AdvanceToNextEvictionTile())
-            return true;
-        }
-        break;
+    iterator_ = TilingData::ReverseSpiralDifferenceIterator(
+        ((*tilings_))[tiling_index_]->tiling_data(),
+        ((*tilings_))[tiling_index_]->current_skewport_rect(),
+        ((*tilings_))[tiling_index_]->current_visible_rect(),
+        ((*tilings_))[tiling_index_]->current_visible_rect());
+    if (!iterator_) {
+      ++tiling_index_;
+      continue;
     }
+    break;
   }
+  if (tiling_index_ >= tilings_->size())
+    return;
+  if (!GetFirstTileAndCheckIfValid(&iterator_))
+    ++(*this);
 }
 
-PictureLayerTilingSet::TilingRange
-TilingSetEvictionQueue::CurrentTilingRange() const {
-  return tiling_set_->GetTilingRange(current_tiling_range_type_);
+TilingSetEvictionQueue::SkewportTilingIterator&
+    TilingSetEvictionQueue::SkewportTilingIterator::
+    operator++() {
+  bool found_tile = AdvanceToNextTile(&iterator_);
+  while (!found_tile && (tiling_index_ + 1) < tilings_->size()) {
+    ++tiling_index_;
+    if (!((*tilings_))[tiling_index_]->has_skewport_rect_tiles())
+      continue;
+    iterator_ = TilingData::ReverseSpiralDifferenceIterator(
+        ((*tilings_))[tiling_index_]->tiling_data(),
+        ((*tilings_))[tiling_index_]->current_skewport_rect(),
+        ((*tilings_))[tiling_index_]->current_visible_rect(),
+        ((*tilings_))[tiling_index_]->current_visible_rect());
+    if (!iterator_)
+      continue;
+    found_tile = GetFirstTileAndCheckIfValid(&iterator_);
+    if (!found_tile)
+      found_tile = AdvanceToNextTile(&iterator_);
+  }
+  return *this;
 }
 
-size_t TilingSetEvictionQueue::CurrentTilingIndex() const {
-  DCHECK_NE(current_tiling_index_, CurrentTilingRange().end);
-  switch (current_tiling_range_type_) {
-    case PictureLayerTilingSet::HIGHER_THAN_HIGH_RES:
-    case PictureLayerTilingSet::LOW_RES:
-    case PictureLayerTilingSet::HIGH_RES:
-      return current_tiling_index_;
-    // Tilings in the following ranges are accessed in reverse order.
-    case PictureLayerTilingSet::BETWEEN_HIGH_AND_LOW_RES:
-    case PictureLayerTilingSet::LOWER_THAN_LOW_RES: {
-      PictureLayerTilingSet::TilingRange tiling_range = CurrentTilingRange();
-      size_t current_tiling_range_offset =
-          current_tiling_index_ - tiling_range.start;
-      return tiling_range.end - 1 - current_tiling_range_offset;
+// VisibleTilingIterator
+TilingSetEvictionQueue::VisibleTilingIterator::VisibleTilingIterator(
+    std::vector<PictureLayerTiling*>* tilings,
+    WhichTree tree,
+    bool skip_shared_out_of_order_tiles,
+    bool return_occluded_tiles,
+    bool return_required_for_activation_tiles)
+    : EvictionRectIterator(tilings, tree, skip_shared_out_of_order_tiles),
+      return_occluded_tiles_(return_occluded_tiles),
+      return_required_for_activation_tiles_(
+          return_required_for_activation_tiles) {
+  // Find the first tiling with a tile.
+  while (tiling_index_ < tilings_->size()) {
+    if (!((*tilings_))[tiling_index_]->has_visible_rect_tiles()) {
+      ++tiling_index_;
+      continue;
+    }
+    iterator_ = TilingData::Iterator(
+        ((*tilings_))[tiling_index_]->tiling_data(),
+        ((*tilings_))[tiling_index_]->current_visible_rect(), false);
+    if (!iterator_) {
+      ++tiling_index_;
+      continue;
     }
+    break;
+  }
+  if (tiling_index_ >= tilings_->size())
+    return;
+  if (!GetFirstTileAndCheckIfValid(&iterator_)) {
+    ++(*this);
+    return;
   }
-  NOTREACHED();
-  return 0;
+  if (!TileMatchesRequiredFlags(tile_)) {
+    ++(*this);
+    return;
+  }
+}
+
+TilingSetEvictionQueue::VisibleTilingIterator&
+    TilingSetEvictionQueue::VisibleTilingIterator::
+    operator++() {
+  bool found_tile = AdvanceToNextTile(&iterator_);
+  while (found_tile && !TileMatchesRequiredFlags(tile_))
+    found_tile = AdvanceToNextTile(&iterator_);
+
+  while (!found_tile && (tiling_index_ + 1) < tilings_->size()) {
+    ++tiling_index_;
+    if (!((*tilings_))[tiling_index_]->has_visible_rect_tiles())
+      continue;
+    iterator_ = TilingData::Iterator(
+        ((*tilings_))[tiling_index_]->tiling_data(),
+        ((*tilings_))[tiling_index_]->current_visible_rect(), false);
+    if (!iterator_)
+      continue;
+    found_tile = GetFirstTileAndCheckIfValid(&iterator_);
+    if (!found_tile)
+      found_tile = AdvanceToNextTile(&iterator_);
+    while (found_tile && !TileMatchesRequiredFlags(tile_))
+      found_tile = AdvanceToNextTile(&iterator_);
+  }
+  return *this;
+}
+
+bool TilingSetEvictionQueue::VisibleTilingIterator::TileMatchesRequiredFlags(
+    const Tile* tile) const {
+  bool activation_flag_matches =
+      tile->required_for_activation() == return_required_for_activation_tiles_;
+  bool occluded_flag_matches =
+      tile->is_occluded(tree_) == return_occluded_tiles_;
+  return activation_flag_matches && occluded_flag_matches;
 }
 
 }  // namespace cc