[#2] Pass gfx structs by const ref (gfx::Rect, gfx::RectF)

cc/trees/quad_culler_unittest.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/trees/quad_culler.h"
 
 #include <vector>
 
 #include "cc/base/math_util.h"
 #include "cc/debug/overdraw_metrics.h"
@@ skipping 12 matching lines @@
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gfx/transform.h"
 
 namespace cc {
 namespace {
 
 class TestOcclusionTrackerImpl
     : public TestOcclusionTrackerBase<LayerImpl, RenderSurfaceImpl> {
  public:
-  TestOcclusionTrackerImpl(gfx::Rect scissor_rect_in_screen,
+  TestOcclusionTrackerImpl(const gfx::Rect& scissor_rect_in_screen,
                            bool record_metrics_for_frame = true)
       : TestOcclusionTrackerBase(scissor_rect_in_screen,
                                  record_metrics_for_frame) {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(TestOcclusionTrackerImpl);
 };
 
 typedef LayerIterator<LayerImpl,
                       LayerImplList,
                       RenderSurfaceImpl,
                       LayerIteratorActions::FrontToBack> LayerIteratorType;
 
 class QuadCullerTest : public testing::Test {
  public:
   QuadCullerTest()
       : host_impl_(&proxy_),
         layer_id_(1) {}
 
   scoped_ptr<TiledLayerImpl> MakeLayer(TiledLayerImpl* parent,
                                        const gfx::Transform& draw_transform,
-                                       gfx::Rect layer_rect,
+                                       const gfx::Rect& layer_rect,
                                        float opacity,
                                        bool opaque,
-                                       gfx::Rect layer_opaque_rect,
+                                       const gfx::Rect& layer_opaque_rect,
                                        LayerImplList& surface_layer_list) {
     scoped_ptr<TiledLayerImpl> layer =
         TiledLayerImpl::Create(host_impl_.active_tree(), layer_id_++);
     scoped_ptr<LayerTilingData> tiler = LayerTilingData::Create(
         gfx::Size(100, 100), LayerTilingData::NO_BORDER_TEXELS);
     tiler->SetBounds(layer_rect.size());
     layer->SetTilingData(*tiler);
     layer->set_skips_draw(false);
     layer->SetDrawsContent(true);
     layer->draw_properties().target_space_transform = draw_transform;
@@ skipping 841 matching lines @@
   EXPECT_EQ(gfx::Rect(15, 10, 5, 11).ToString(),
             quad_list[0]->visible_rect.ToString());
   EXPECT_EQ(gfx::Rect(15, 20, 8, 14).ToString(),
             quad_list[1]->visible_rect.ToString());
   EXPECT_EQ(gfx::Rect(15, 30, 10, 16).ToString(),
             quad_list[2]->visible_rect.ToString());
 }
 
 }  // namespace
 }  // namespace cc