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

Unified Diff: ui/native_theme/common_theme.cc

Issue 2680943002: ui: Clean up naming of paint-related identifiers (Closed)
Patch Set: Created 3 years, 10 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
« no previous file with comments | « ui/gfx/render_text_unittest.cc ('k') | ui/native_theme/native_theme_aura.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/native_theme/common_theme.cc
diff --git a/ui/native_theme/common_theme.cc b/ui/native_theme/common_theme.cc
index 861e4e1915b422910d627c9919ac277e259eb5a1..021abe1f7bb207ff89b1a5cbf8865a1b6c43fa8f 100644
--- a/ui/native_theme/common_theme.cc
+++ b/ui/native_theme/common_theme.cc
@@ -344,15 +344,15 @@ void CommonThemePaintMenuItemBackground(
NativeTheme::State state,
const gfx::Rect& rect,
const NativeTheme::MenuItemExtraParams& menu_item) {
- cc::PaintFlags paint;
+ cc::PaintFlags flags;
switch (state) {
case NativeTheme::kNormal:
case NativeTheme::kDisabled:
- paint.setColor(
+ flags.setColor(
theme->GetSystemColor(NativeTheme::kColorId_MenuBackgroundColor));
break;
case NativeTheme::kHovered:
- paint.setColor(theme->GetSystemColor(
+ flags.setColor(theme->GetSystemColor(
NativeTheme::kColorId_FocusedMenuItemBackgroundColor));
break;
default:
@@ -361,10 +361,10 @@ void CommonThemePaintMenuItemBackground(
}
if (menu_item.corner_radius > 0) {
const SkScalar radius = SkIntToScalar(menu_item.corner_radius);
- canvas->drawRoundRect(gfx::RectToSkRect(rect), radius, radius, paint);
+ canvas->drawRoundRect(gfx::RectToSkRect(rect), radius, radius, flags);
return;
}
- canvas->drawRect(gfx::RectToSkRect(rect), paint);
+ canvas->drawRect(gfx::RectToSkRect(rect), flags);
}
} // namespace ui
« no previous file with comments | « ui/gfx/render_text_unittest.cc ('k') | ui/native_theme/native_theme_aura.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698