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

content/renderer/gpu/compositor_software_output_device.h

 // Copyright (c) 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.
 
 #ifndef CONTENT_RENDERER_GPU_COMPOSITOR_SOFTWARE_OUTPUT_DEVICE_H_
 #define CONTENT_RENDERER_GPU_COMPOSITOR_SOFTWARE_OUTPUT_DEVICE_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "base/memory/shared_memory.h"
@@ skipping 10 matching lines @@
 // to a fixed thread when BindToClient is called.
 class CompositorSoftwareOutputDevice
     : NON_EXPORTED_BASE(public cc::SoftwareOutputDevice),
       NON_EXPORTED_BASE(public base::NonThreadSafe) {
 public:
   CompositorSoftwareOutputDevice();
   virtual ~CompositorSoftwareOutputDevice();
 
   virtual void Resize(gfx::Size size) OVERRIDE;
 
-  virtual SkCanvas* BeginPaint(gfx::Rect damage_rect) OVERRIDE;
+  virtual SkCanvas* BeginPaint(const gfx::Rect& damage_rect) OVERRIDE;
   virtual void EndPaint(cc::SoftwareFrameData* frame_data) OVERRIDE;
   virtual void EnsureBackbuffer() OVERRIDE;
   virtual void DiscardBackbuffer() OVERRIDE;
 
   virtual void ReclaimSoftwareFrame(unsigned id) OVERRIDE;
 
 private:
   // Internal buffer class that manages shared memory lifetime and ownership.
   // It also tracks buffers' history so we can calculate what's the minimum
   // damage rect difference between any two given buffers (see SetParent and
@@ skipping 47 matching lines @@
   unsigned next_buffer_id_;
   ScopedVector<Buffer> buffers_;
   ScopedVector<Buffer> awaiting_ack_;
   SkBitmap bitmap_;
   RenderThread* render_thread_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_GPU_COMPOSITOR_SOFTWARE_OUTPUT_DEVICE_H_