Chromium Code Reviews| Index: ui/views/controls/menu/menu_delegate.h |
| diff --git a/ui/views/controls/menu/menu_delegate.h b/ui/views/controls/menu/menu_delegate.h |
| index e50ec8e5ce0d6c28865cf6954288470d9c4215f2..2540e0e3d0828585f45e152d47c8f0f2fb2825fc 100644 |
| --- a/ui/views/controls/menu/menu_delegate.h |
| +++ b/ui/views/controls/menu/menu_delegate.h |
| @@ -66,9 +66,15 @@ class VIEWS_EXPORT MenuDelegate { |
| // The font for the menu item label. |
| virtual const gfx::FontList* GetLabelFontList(int id) const; |
| + // Whether this item should be displayed with a bolder color when disabled. |
| + virtual bool GetBoldedDisabled(int command_id) const; |
|
sky
2014/03/20 02:07:49
Bold implies a font. How about GetShouldUseDisable
|
| + |
| // Override the text color of a given menu item dependent on the |
| // |command_id| and its |is_hovered| state. Returns true if it chooses to |
| // override the color. |
| + // |
| + // TODO(erg): Remove this interface. Injecting raw colors into the menu |
| + // circumvents the NativeTheme. |
| virtual bool GetForegroundColor(int command_id, |
| bool is_hovered, |
| SkColor* override_color) const; |
| @@ -76,6 +82,9 @@ class VIEWS_EXPORT MenuDelegate { |
| // Override the background color of a given menu item dependent on the |
| // |command_id| and its |is_hovered| state. Returns true if it chooses to |
| // override the color. |
| + // |
| + // TODO(erg): Remove this interface. Injecting raw colors into the menu |
| + // circumvents the NativeTheme. |
| virtual bool GetBackgroundColor(int command_id, |
| bool is_hovered, |
| SkColor* override_color) const; |