cc: Remove vectors from tiling eviction tile iterator.

cc/layers/picture_layer_impl.h

 // 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.
 
 #ifndef CC_LAYERS_PICTURE_LAYER_IMPL_H_
 #define CC_LAYERS_PICTURE_LAYER_IMPL_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 59 matching lines @@
     LayerEvictionTileIterator(PictureLayerImpl* layer,
                               TreePriority tree_priority);
     ~LayerEvictionTileIterator();
 
     Tile* operator*();
     const Tile* operator*() const;
     LayerEvictionTileIterator& operator++();
     operator bool() const;
 
    private:
+    // The order of this enum determines the order of tiling iteration.
+    enum TilingIterationDirection {
+      HIGHER_THAN_HIGH_RES,
+      LOWER_THAN_HIGH_RES,
+      LOW_RES,
+      HIGH_RES,
+      NUM_ITERATION_DIRECTIONS
+    };

[reveman, 2014/07/29 01:24:25]

Can you describe all the phases that this iterator goes through. Is it each one
of these directions for each iteration stage? Btw, why is it called a direction?
rather then say a phase, or stage?

+
     void AdvanceToNextIterator();
-    bool IsCorrectType(
-        PictureLayerTiling::TilingEvictionTileIterator* it) const;
+    bool AdvanceNextTilingIndex();
 
-    std::vector<PictureLayerTiling::TilingEvictionTileIterator> iterators_;
-    size_t iterator_index_;
+    PictureLayerTiling::TilingEvictionTileIterator iterator_;
+    size_t tiling_index_;
+    TilingIterationDirection tiling_direction_;
     TilePriority::PriorityBin iteration_stage_;
     bool required_for_activation_;
+    size_t high_res_tiling_index_;
+    size_t low_res_tiling_index_;
+    TreePriority tree_priority_;
 
     PictureLayerImpl* layer_;
   };
 
   static scoped_ptr<PictureLayerImpl> Create(LayerTreeImpl* tree_impl, int id) {
     return make_scoped_ptr(new PictureLayerImpl(tree_impl, id));
   }
   virtual ~PictureLayerImpl();
 
   // LayerImpl overrides.
@@ skipping 119 matching lines @@
   gfx::Size viewport_size_for_tile_priority_;
   gfx::Transform screen_space_transform_for_tile_priority_;
 
   friend class PictureLayer;
   DISALLOW_COPY_AND_ASSIGN(PictureLayerImpl);
 };
 
 }  // namespace cc
 
 #endif  // CC_LAYERS_PICTURE_LAYER_IMPL_H_