cc: Clean up iterator template to only take 1 parameter.

cc/debug/debug_rect_history.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/debug/debug_rect_history.h"
 
 #include "cc/base/math_util.h"
 #include "cc/layers/layer_impl.h"
 #include "cc/layers/layer_utils.h"
 #include "cc/layers/render_surface_impl.h"
@@ skipping 94 matching lines @@
       if (LayerTreeHostCommon::RenderSurfaceContributesToTarget<LayerImpl>(
               layer, render_surface_layer->id()))
         continue;
 
       if (layer->LayerIsAlwaysDamaged())
         continue;
 
       if (layer->LayerPropertyChanged()) {
         debug_rects_.push_back(
             DebugRect(PROPERTY_CHANGED_RECT_TYPE,
-                      MathUtil::MapEnclosingClippedRect(
+                      MathUtil::MapClippedRect(

[enne (OOO), 2014/02/11 23:25:22]

This is just a drive-by? Maybe stick it in a different patch?

[danakj, 2014/02/11 23:31:00]

This was not meant to be here.. it was a mistake in my other CL that I fixed.
Thanks for spotting it.

                           layer->screen_space_transform(),
-                          gfx::Rect(layer->content_bounds()))));
+                          gfx::RectF(gfx::PointF(), layer->content_bounds()))));
       }
     }
   }
 }
 
 void DebugRectHistory::SaveSurfaceDamageRects(
     const LayerImplList& render_surface_layer_list) {
   for (int surface_index = render_surface_layer_list.size() - 1;
        surface_index >= 0;
        --surface_index) {
@@ skipping 103 matching lines @@
                                             layer->contents_scale_y());
     debug_rects_.push_back(DebugRect(NON_FAST_SCROLLABLE_RECT_TYPE,
                                      MathUtil::MapClippedRect(
                                          layer->screen_space_transform(),
                                          scroll_rect)));
   }
 }
 
 void DebugRectHistory::SaveLayerAnimationBoundsRects(
     const LayerImplList& render_surface_layer_list) {
-  typedef LayerIterator<LayerImpl,
-                        LayerImplList,
-                        RenderSurfaceImpl,
-                        LayerIteratorActions::FrontToBack> LayerIteratorType;
+  typedef LayerIterator<LayerImpl> LayerIteratorType;
   LayerIteratorType end = LayerIteratorType::End(&render_surface_layer_list);
   for (LayerIteratorType it =
            LayerIteratorType::Begin(&render_surface_layer_list);
        it != end; ++it) {
     if (!it.represents_itself())
       continue;
 
     // TODO(avallee): Figure out if we should show something for a layer who's
     // animating bounds but that we can't compute them.
     gfx::BoxF inflated_bounds;
     if (!LayerUtils::GetAnimationBounds(**it, &inflated_bounds))
       continue;
 
     debug_rects_.push_back(DebugRect(ANIMATION_BOUNDS_RECT_TYPE,
                                      gfx::RectF(inflated_bounds.x(),
                                                 inflated_bounds.y(),
                                                 inflated_bounds.width(),
                                                 inflated_bounds.height())));
   }
 }
 
 }  // namespace cc