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

cc/test/layer_tree_pixel_test.cc

 // Copyright 2013 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/test/layer_tree_pixel_test.h"
 
 #include "base/command_line.h"
 #include "base/path_service.h"
 #include "cc/base/switches.h"
 #include "cc/layers/solid_color_layer.h"
@@ skipping 101 matching lines @@
   CommandLine* cmd = CommandLine::ForCurrentProcess();
   if (cmd->HasSwitch(switches::kCCRebaselinePixeltests))
     EXPECT_TRUE(WritePNGFile(*result_bitmap_, ref_file_path, true));
 
   EXPECT_TRUE(MatchesPNGFile(*result_bitmap_,
                              ref_file_path,
                              *pixel_comparator_));
 }
 
 scoped_refptr<SolidColorLayer> LayerTreePixelTest::CreateSolidColorLayer(
-    gfx::Rect rect, SkColor color) {
+    const gfx::Rect& rect, SkColor color) {
   scoped_refptr<SolidColorLayer> layer = SolidColorLayer::Create();
   layer->SetIsDrawable(true);
   layer->SetAnchorPoint(gfx::PointF());
   layer->SetBounds(rect.size());
   layer->SetPosition(rect.origin());
   layer->SetBackgroundColor(color);
   return layer;
 }
 
 void LayerTreePixelTest::EndTest() {
@@ skipping 10 matching lines @@
 void LayerTreePixelTest::TryEndTest() {
   if (!result_bitmap_)
     return;
   if (pending_texture_mailbox_callbacks_)
     return;
   LayerTreeTest::EndTest();
 }
 
 scoped_refptr<SolidColorLayer> LayerTreePixelTest::
     CreateSolidColorLayerWithBorder(
-        gfx::Rect rect, SkColor color, int border_width, SkColor border_color) {
+        const gfx::Rect& rect, SkColor color,
+        int border_width, SkColor border_color) {
   scoped_refptr<SolidColorLayer> layer = CreateSolidColorLayer(rect, color);
   scoped_refptr<SolidColorLayer> border_top = CreateSolidColorLayer(
       gfx::Rect(0, 0, rect.width(), border_width), border_color);
   scoped_refptr<SolidColorLayer> border_left = CreateSolidColorLayer(
       gfx::Rect(0,
                 border_width,
                 border_width,
                 rect.height() - border_width * 2),
       border_color);
   scoped_refptr<SolidColorLayer> border_right =
       CreateSolidColorLayer(gfx::Rect(rect.width() - border_width,
                                       border_width,
                                       border_width,
                                       rect.height() - border_width * 2),
                             border_color);
   scoped_refptr<SolidColorLayer> border_bottom = CreateSolidColorLayer(
       gfx::Rect(0, rect.height() - border_width, rect.width(), border_width),
       border_color);
   layer->AddChild(border_top);
   layer->AddChild(border_left);
   layer->AddChild(border_right);
   layer->AddChild(border_bottom);
   return layer;
 }
 
 scoped_refptr<TextureLayer> LayerTreePixelTest::CreateTextureLayer(
-    gfx::Rect rect, const SkBitmap& bitmap) {
+    const gfx::Rect& rect, const SkBitmap& bitmap) {
   scoped_refptr<TextureLayer> layer = TextureLayer::CreateForMailbox(NULL);
   layer->SetIsDrawable(true);
   layer->SetAnchorPoint(gfx::PointF());
   layer->SetBounds(rect.size());
   layer->SetPosition(rect.origin());
 
   TextureMailbox texture_mailbox;
   scoped_ptr<SingleReleaseCallback> release_callback;
   CopyBitmapToTextureMailboxAsTexture(
       bitmap, &texture_mailbox, &release_callback);
@@ skipping 179 matching lines @@
 
   *texture_mailbox = TextureMailbox(mailbox, sync_point);
   *release_callback = SingleReleaseCallback::Create(
       base::Bind(&LayerTreePixelTest::ReleaseTextureMailbox,
                  base::Unretained(this),
                  base::Passed(&context),
                  texture_id));
 }
 
 }  // namespace cc