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

chrome/browser/ui/views/toolbar_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 "chrome/browser/ui/views/toolbar_view.h"
 
 #include "base/i18n/number_formatting.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/prefs/pref_service.h"
@@ skipping 565 matching lines @@
     canvas->DrawBitmapInt(*kPopupBackgroundEdge, 0, top_spacing);
     canvas->DrawBitmapInt(*kPopupBackgroundEdge,
                           width() - kPopupBackgroundEdge->width(), top_spacing);
   }
 
   // For glass, we need to draw a black line below the location bar to separate
   // it from the content area.  For non-glass, the NonClientView draws the
   // toolbar background below the location bar for us.
   // NOTE: Keep this in sync with BrowserView::GetInfoBarSeparatorColor()!
   if (GetWidget()->ShouldUseNativeFrame())
-    canvas->FillRect(SK_ColorBLACK, gfx::Rect(0, height() - 1, width(), 1));
+    canvas->FillRect(gfx::Rect(0, height() - 1, width(), 1), SK_ColorBLACK);
 }
 
 // Note this method is ignored on Windows, but needs to be implemented for
 // linux, where it is called before CanDrop().
 bool ToolbarView::GetDropFormats(
     int* formats,
     std::set<OSExchangeData::CustomFormat>* custom_formats) {
   *formats = ui::OSExchangeData::URL | ui::OSExchangeData::STRING;
   return true;
 }
@@ skipping 135 matching lines @@
     accname_app = l10n_util::GetStringFUTF16(
         IDS_ACCNAME_APP_UPGRADE_RECOMMENDED, accname_app);
   }
   app_menu_->SetAccessibleName(accname_app);
 
   app_menu_->SetIcon(GetAppMenuIcon(views::CustomButton::BS_NORMAL));
   app_menu_->SetHoverIcon(GetAppMenuIcon(views::CustomButton::BS_HOT));
   app_menu_->SetPushedIcon(GetAppMenuIcon(views::CustomButton::BS_PUSHED));
   SchedulePaint();
 }