From d4931a7e37d339e526421b1c4ef662543d986a53 Mon Sep 17 00:00:00 2001 From: "tfarina@chromium.org" Date: Thu, 1 Mar 2012 21:57:43 +0000 Subject: ui/gfx: Drop unused args from PaintMenuBackgroundColor() and pass gfx::Size instead of gfx::Rect. This is a follow up patch to address the comments from: http://codereview.chromium.org/9544001/diff/2007/ui/gfx/native_theme_chromeos.cc#newcode662 R=pkasting@chromium.org TBR=sky@chromium.org Review URL: https://chromiumcodereview.appspot.com/9565021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124493 0039d316-1c4b-4281-b951-d872f2087c98 --- ui/gfx/native_theme_base.cc | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) (limited to 'ui/gfx/native_theme_base.cc') diff --git a/ui/gfx/native_theme_base.cc b/ui/gfx/native_theme_base.cc index 0355631..8e5871b 100644 --- a/ui/gfx/native_theme_base.cc +++ b/ui/gfx/native_theme_base.cc @@ -145,7 +145,7 @@ void NativeThemeBase::Paint(SkCanvas* canvas, NOTIMPLEMENTED(); break; case kMenuPopupBackground: - PaintMenuPopupBackground(canvas, state, rect, extra.menu_list); + PaintMenuPopupBackground(canvas, rect.size()); break; case kMenuPopupGutter: case kMenuPopupSeparator: @@ -666,11 +666,8 @@ void NativeThemeBase::PaintMenuList( canvas->drawPath(path, paint); } -void NativeThemeBase::PaintMenuPopupBackground( - SkCanvas* canvas, - State state, - const gfx::Rect& rect, - const MenuListExtraParams& menu_list) const { +void NativeThemeBase::PaintMenuPopupBackground(SkCanvas* canvas, + const gfx::Size& size) const { canvas->drawColor(kMenuPopupBackgroundColor, SkXfermode::kSrc_Mode); } -- cgit v1.1