Pass const gfx::Rect& as the first parameter to FillRect.

ash/wm/toplevel_frame_view.cc

 // Copyright (c) 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 "ash/wm/toplevel_frame_view.h"
 
 #include "grit/ui_resources.h"
 #include "ui/aura/cursor.h"
 #include "ui/base/animation/throb_animation.h"
 #include "ui/base/hit_test.h"
@@ skipping 25 matching lines @@
                       SkColor color,
                       const SkBitmap& icon)
       : views::CustomButton(listener),
         color_(color),
         icon_(icon) {
   }
   virtual ~WindowControlButton() {}
 
   // Overridden from views::View:
   virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE {
-    canvas->FillRect(GetBackgroundColor(), GetLocalBounds());
+    canvas->FillRect(GetLocalBounds(), GetBackgroundColor());
     canvas->DrawBitmapInt(icon_, 0, 0);
   }
   virtual gfx::Size GetPreferredSize() OVERRIDE {
     gfx::Size size(icon_.width(), icon_.height());
     size.Enlarge(3, 2);
     return size;
   }
 
  private:
   SkColor GetBackgroundColor() {
@@ skipping 49 matching lines @@
       : ALLOW_THIS_IN_INITIALIZER_LIST(
             animation_(new ui::SlideAnimation(this))) {
     animation_->SetSlideDuration(kHoverFadeDurationMs);
   }
 
   // Most of the frame components are rendered with a transparent bg.
   void PaintTransparentBackground(gfx::Canvas* canvas) {
     // Fill with current opacity value.
     int opacity = animation_->CurrentValueBetween(kFrameBorderHiddenOpacity,
                                                   kFrameBorderVisibleOpacity);
-    canvas->FillRect(SkColorSetARGB(opacity,
+    canvas->FillRect(GetLocalBounds(),
+                     SkColorSetARGB(opacity,
                                     SkColorGetR(kFrameColor),
                                     SkColorGetG(kFrameColor),
-                                    SkColorGetB(kFrameColor)),
-                     GetLocalBounds());
+                                    SkColorGetB(kFrameColor)));
   }
 
   // Overridden from ui::AnimationDelegate:
   virtual void AnimationProgressed(const ui::Animation* animation) OVERRIDE {
     SchedulePaint();
   }
 
  private:
   scoped_ptr<ui::SlideAnimation> animation_;
 
@@ skipping 347 matching lines @@
       return aura::kCursorNorthWestResize;
     case HTTOPRIGHT:
       return aura::kCursorNorthEastResize;
     default:
       return aura::kCursorNull;
   }
 }
 
 }  // namespace internal
 }  // namespace ash