Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(58)

Unified Diff: chrome/browser/ui/views/frame/app_panel_browser_frame_view.cc

Issue 9021046: Pass const gfx::Rect& as the first parameter to FillRect. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: one more fix Created 8 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/frame/app_panel_browser_frame_view.cc
diff --git a/chrome/browser/ui/views/frame/app_panel_browser_frame_view.cc b/chrome/browser/ui/views/frame/app_panel_browser_frame_view.cc
index d8670731739007a2cc22df1b88789b5b3b0ac709..4226f7a9a1703f93ae7157619d3912473454cfcd 100644
--- a/chrome/browser/ui/views/frame/app_panel_browser_frame_view.cc
+++ b/chrome/browser/ui/views/frame/app_panel_browser_frame_view.cc
@@ -335,26 +335,23 @@ void AppPanelBrowserFrameView::PaintRestoredFrameBorder(gfx::Canvas* canvas) {
// Fill with the frame color first so we have a constant background for
// areas not covered by the theme image.
- canvas->FillRect(frame_color,
- gfx::Rect(0, 0, width(), theme_frame->height()));
+ canvas->FillRect(gfx::Rect(0, 0, width(), theme_frame->height()),
+ frame_color);
int remaining_height = height() - theme_frame->height();
if (remaining_height > 0) {
// Now fill down the sides.
- canvas->FillRect(frame_color,
- gfx::Rect(0, theme_frame->height(), left_edge->width(),
- remaining_height));
- canvas->FillRect(frame_color,
- gfx::Rect(width() - right_edge->width(),
+ canvas->FillRect(gfx::Rect(0, theme_frame->height(), left_edge->width(),
+ remaining_height), frame_color);
+ canvas->FillRect(gfx::Rect(width() - right_edge->width(),
theme_frame->height(), right_edge->width(),
- remaining_height));
+ remaining_height), frame_color);
int center_width = width() - left_edge->width() - right_edge->width();
if (center_width > 0) {
// Now fill the bottom area.
- canvas->FillRect(frame_color,
- gfx::Rect(left_edge->width(),
- height() - bottom_edge->height(),
- center_width, bottom_edge->height()));
+ canvas->FillRect(gfx::Rect(left_edge->width(),
+ height() - bottom_edge->height(), center_width,
+ bottom_edge->height()), frame_color);
}
}
« no previous file with comments | « chrome/browser/ui/views/download/download_shelf_view.cc ('k') | chrome/browser/ui/views/frame/browser_frame_aura.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698