Chromium Code Reviews| 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 cd7a352f8fff8db59f35f126236e744dda4e2dd7..5097c4852f9e9e13f9af85f404063e9a67afcf78 100644 |
| --- a/ui/views/controls/menu/menu_image_util.h |
| +++ b/ui/views/controls/menu/menu_image_util.h |
| @@ -17,10 +17,13 @@ const int kSubmenuArrowSize = 8; |
| // Returns the Menu Check box image (always checked). |
| gfx::ImageSkia GetMenuCheckImage(SkColor icon_color); |
| -// Return the RadioButton image for given state. |
| -// It returns the "selected" image when |selected| is |
| -// true, or the "unselected" image if false. |
| -gfx::ImageSkia GetRadioButtonImage(bool selected); |
| +// Return the RadioButton image for given state. |toggled| is true when |
| +// the radio option is active, |hovered| describes the menu higlight/selection |
|
sadrul
2016/01/26 21:59:30
"the radio option is active" would 'selected' be b
Evan Stade
2016/01/26 22:31:53
I avoided the word selection because it's ambiguou
|
| +// state, and |default_icon_color| is the base color that should be used for |
| +// the icon (which may be ignored based on the other two flags). |
| +gfx::ImageSkia GetRadioButtonImage(bool toggled, |
| + bool hovered, |
| + SkColor default_icon_color); |
| // Returns the image for submenu arrow for current RTL setting. |
| gfx::ImageSkia GetSubmenuArrowImage(SkColor icon_color); |