cc: Start using raster/eviction iterators.

cc/resources/picture_layer_tiling_set.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/picture_layer_tiling_set.h"
 
 #include <limits>
 
 namespace cc {
 
@@ skipping 200 matching lines @@
     return NULL;
   return *tiling_iter_;
 }
 
 Tile* PictureLayerTilingSet::CoverageIterator::operator*() const {
   if (!tiling_iter_)
     return NULL;
   return *tiling_iter_;
 }
 
-PictureLayerTiling* PictureLayerTilingSet::CoverageIterator::CurrentTiling() {
+TileResolution PictureLayerTilingSet::CoverageIterator::resolution() const {
+  const PictureLayerTiling* tiling = CurrentTiling();
+  if (tiling)

[danakj, 2014/10/08 18:46:26]

this is a case of doing if() for something you also DCHECK() (or NOTREACHED in
this case).

just DCHECK(tiling) if you wanna, and deref it.

[vmpstr, 2014/10/08 22:38:34]

Done.

+    return tiling->resolution();
+  NOTREACHED();
+  return NON_IDEAL_RESOLUTION;
+}
+
+PictureLayerTiling* PictureLayerTilingSet::CoverageIterator::CurrentTiling()
+    const {
   if (current_tiling_ < 0)
     return NULL;
   if (static_cast<size_t>(current_tiling_) >= set_->tilings_.size())
     return NULL;
   return set_->tilings_[current_tiling_];
 }
 
 int PictureLayerTilingSet::CoverageIterator::NextTiling() const {
   // Order returned by this method is:
   // 1. Ideal tiling index
@@ skipping 68 matching lines @@
   }
 
   return *this;
 }
 
 PictureLayerTilingSet::CoverageIterator::operator bool() const {
   return current_tiling_ < static_cast<int>(set_->tilings_.size()) ||
       region_iter_.has_rect();
 }
 
-void PictureLayerTilingSet::DidBecomeActive() {
-  for (size_t i = 0; i < tilings_.size(); ++i)
-    tilings_[i]->DidBecomeActive();
-}
-
-void PictureLayerTilingSet::DidBecomeRecycled() {
-  for (size_t i = 0; i < tilings_.size(); ++i)
-    tilings_[i]->DidBecomeRecycled();
-}
-
 void PictureLayerTilingSet::AsValueInto(base::debug::TracedValue* state) const {
   for (size_t i = 0; i < tilings_.size(); ++i) {
     state->BeginDictionary();
     tilings_[i]->AsValueInto(state);
     state->EndDictionary();
   }
 }
 
 size_t PictureLayerTilingSet::GPUMemoryUsageInBytes() const {
   size_t amount = 0;
@@ skipping 28 matching lines @@
       return low_res_range;
     case LOWER_THAN_LOW_RES:
       return TilingRange(low_res_range.end, tilings_.size());
   }
 
   NOTREACHED();
   return TilingRange(0, 0);
 }
 
 }  // namespace cc