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

Unified Diff: ui/views/controls/menu/menu_image_util.h

Issue 25144002: views: Make sure menu checks and submenu arrows have high contrast even when selected. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync Created 7 years, 3 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/views/controls/menu/menu_config_views.cc ('k') | ui/views/controls/menu/menu_image_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/controls/menu/menu_image_util.h
diff --git a/ui/views/controls/menu/menu_image_util.h b/ui/views/controls/menu/menu_image_util.h
index cf8c8db092ef112b9c90ec583894fde49dab77c4..93645c8d7e90ec31a3cc16de4406607276969644 100644
--- a/ui/views/controls/menu/menu_image_util.h
+++ b/ui/views/controls/menu/menu_image_util.h
@@ -13,16 +13,20 @@ namespace views {
// Returns the Menu Check box image (always checked).
// The returned image is global object and should not be freed.
-const gfx::ImageSkia* GetMenuCheckImage();
+// |dark_background| should be true if the check will be displayed on a
+// dark background (such as a hovered menu item).
+gfx::ImageSkia GetMenuCheckImage(bool dark_background);
// Return the RadioButton image for given state.
// It returns the "selected" image when |selected| is
// true, or the "unselected" image if false.
// The returned image is global object and should not be freed.
-const gfx::ImageSkia* GetRadioButtonImage(bool selected);
+gfx::ImageSkia GetRadioButtonImage(bool selected);
// Returns the image for submenu arrow for current RTL setting.
-const gfx::ImageSkia* GetSubmenuArrowImage();
+// |dark_background| should be true if the check will be displayed on a
+// dark background (such as a hovered menu item).
+gfx::ImageSkia GetSubmenuArrowImage(bool dark_background);
} // namespace views
« no previous file with comments | « ui/views/controls/menu/menu_config_views.cc ('k') | ui/views/controls/menu/menu_image_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698