GitLab

This change attempts to be a behavior preserving change at the views
layer while allowing the Chrome Typography provider to specify colors
for menu item text.

The mapping of Native Color IDs to styles are as follows:

kColorId_EnabledMenuItemForegroundColor     = CONTEXT_MENU + STYLE_PRIMARY
kColorId_SelectedMenuItemForegroundColor    = CONTEXT_MENU + STYLE_SELECTED
kColorId_DisabledMenuItemForegroundColor    = CONTEXT_MENU + STYLE_DISABLED
kColorId_TouchableMenuItemLabelColor        = CONTEXT_TOUCH_MENU + STYLE_PRIMARY
kColorId_MenuItemMinorTextColor             = CONTEXT_MENU + STYLE_SECONDARY
kColorId_HighlightedMenuItemForegroundColor = CONTEXT_MENU + STYLE_HIGHLIGHTED

The resultant colors are as follows, with views representing the old colors.

                                             Light Mode       Light Mode
                                             Default (Views)  Chrome
kColorId_EnabledMenuItemForegroundColor      Black            GG900
kColorId_SelectedMenuItemForegroundColor     Black            GG900
kColorId_DisabledMenuItemForegroundColor     GG600 [1]        9e9e9e
kColorId_TouchableMenuItemLabelColor         GG900            GG900
kColorId_MenuItemMinorTextColor              Black A:0x89     GG700
kColorId_HighlightedMenuItemForegroundColor  GG900            GG900
* (A:Value) means Alpha Set to Value, transparent at 0, fully opaque at 255.
[1] kDisabledTextColor

                                             Dark Mode        Dark Mode
                                             Default (Views)  Chrome
kColorId_EnabledMenuItemForegroundColor      GG200            White A:0xDD
kColorId_SelectedMenuItemForegroundColor     GG200            White A:0xDD
kColorId_DisabledMenuItemForegroundColor     GG600            GG600 [2]
kColorId_TouchableMenuItemLabelColor         GG900            White A:0xDD
kColorId_MenuItemMinorTextColor              Black A:0x89     GG500
kColorId_HighlightedMenuItemForegroundColor  GG200            White A:0xDD
* (A:Value) means Alpha Set to Value, transparent at 0, fully opaque at 255.
[2] Was GG800 by default, but changed to GG600 to maintain previous contrast.

We can consider merging the colors after this change.

BUG=865318

Change-Id: Ie0c0b566637e752e9947c702e7fb8160d51a3558
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1767083
Commit-Queue: Robert Liao <robliao@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#694284}