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

cc/trees/thread_proxy.cc

 // Copyright 2011 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/thread_proxy.h"
 
 #include <string>
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
@@ skipping 108 matching lines @@
   DCHECK(IsMainThread());
   DCHECK(this->layer_tree_host());
 }
 
 ThreadProxy::~ThreadProxy() {
   TRACE_EVENT0("cc", "ThreadProxy::~ThreadProxy");
   DCHECK(IsMainThread());
   DCHECK(!started_);
 }
 
-bool ThreadProxy::CompositeAndReadback(void* pixels, gfx::Rect rect) {
+bool ThreadProxy::CompositeAndReadback(void* pixels, const gfx::Rect& rect) {
   TRACE_EVENT0("cc", "ThreadProxy::CompositeAndReadback");
   DCHECK(IsMainThread());
   DCHECK(layer_tree_host());
 
   if (defer_commits_) {
     TRACE_EVENT0("cc", "CompositeAndReadback_DeferCommit");
     return false;
   }
 
   if (!layer_tree_host()->InitializeOutputSurfaceIfNeeded()) {
@@ skipping 360 matching lines @@
 
   layer_tree_host_impl_->SendManagedMemoryStats(
       contents_texture_manager_on_impl_thread()->MemoryVisibleBytes(),
       contents_texture_manager_on_impl_thread()->
           MemoryVisibleAndNearbyBytes(),
       contents_texture_manager_on_impl_thread()->MemoryUseBytes());
 }
 
 bool ThreadProxy::IsInsideDraw() { return inside_draw_; }
 
-void ThreadProxy::SetNeedsRedraw(gfx::Rect damage_rect) {
+void ThreadProxy::SetNeedsRedraw(const gfx::Rect& damage_rect) {
   DCHECK(IsMainThread());
   TRACE_EVENT0("cc", "ThreadProxy::SetNeedsRedraw");
   Proxy::ImplThreadTaskRunner()->PostTask(
       FROM_HERE,
       base::Bind(&ThreadProxy::SetNeedsRedrawRectOnImplThread,
                  impl_thread_weak_ptr_,
                  damage_rect));
 }
 
 void ThreadProxy::SetNextCommitWaitsForActivation() {
@@ skipping 34 matching lines @@
   DCHECK(IsImplThread());
   TRACE_EVENT0("cc", "ThreadProxy::SetNeedsRedrawOnImplThread");
   scheduler_on_impl_thread_->SetNeedsRedraw();
 }
 
 void ThreadProxy::SetNeedsManageTilesOnImplThread() {
   DCHECK(IsImplThread());
   scheduler_on_impl_thread_->SetNeedsManageTiles();
 }
 
-void ThreadProxy::SetNeedsRedrawRectOnImplThread(gfx::Rect damage_rect) {
+void ThreadProxy::SetNeedsRedrawRectOnImplThread(const gfx::Rect& damage_rect) {
   DCHECK(IsImplThread());
   layer_tree_host_impl_->SetViewportDamage(damage_rect);
   SetNeedsRedrawOnImplThread();
 }
 
 void ThreadProxy::SetSwapUsedIncompleteTileOnImplThread(
     bool used_incomplete_tile) {
   DCHECK(IsImplThread());
   if (used_incomplete_tile) {
     TRACE_EVENT_INSTANT0(
@@ skipping 1066 matching lines @@
   commit_to_activate_duration_history_.InsertSample(
       base::TimeTicks::HighResNow() - commit_complete_time_);
 }
 
 void ThreadProxy::DidManageTiles() {
   DCHECK(IsImplThread());
   scheduler_on_impl_thread_->DidManageTiles();
 }
 
 }  // namespace cc