MenuItemView: use disabled text color for subtitles too

chrome/browser/ui/libgtkui/native_theme_gtk3.cc

Index: chrome/browser/ui/libgtkui/native_theme_gtk3.cc
diff --git a/chrome/browser/ui/libgtkui/native_theme_gtk3.cc b/chrome/browser/ui/libgtkui/native_theme_gtk3.cc
index cae8210567310422c45f3f9273730e01c8f4b70b..d0a7dbdad724d7d06e1fce6d9cc6775909165565 100644
--- a/chrome/browser/ui/libgtkui/native_theme_gtk3.cc
+++ b/chrome/browser/ui/libgtkui/native_theme_gtk3.cc
@@ -112,6 +112,9 @@ SkColor SkColorFromColorId(ui::NativeTheme::ColorId color_id) {
     case ui::NativeTheme::kColorId_DisabledMenuItemForegroundColor:
       return GetFgColor("GtkMenu#menu GtkMenuItem#menuitem:disabled GtkLabel");
     case ui::NativeTheme::kColorId_MenuItemSubtitleColor:
+      if (GtkVersionCheck(3, 20)) {
+        return GetFgColor("GtkMenu#menu GtkMenuItem#menuitem #accelerator");

[Tom (Use chromium acct), 2017/02/15 21:33:18]

This was actually fine before, but this is more technically correct

[Elliot Glaysher, 2017/02/15 21:48:19]

The best kind of correct!

+      }
       return GetFgColor(
           "GtkMenu#menu GtkMenuItem#menuitem GtkLabel.accelerator");
     case ui::NativeTheme::kColorId_MenuSeparatorColor:
@@ -122,9 +125,8 @@ SkColor SkColorFromColorId(ui::NativeTheme::ColorId color_id) {
       if (GtkVersionCheck(3, 20)) {
         return GetSeparatorColor(
             "GtkMenu#menu GtkSeparator#separator.horizontal");
-      } else {
-        return GetFgColor("GtkMenu#menu GtkMenuItem#menuitem.separator");
       }
+      return GetFgColor("GtkMenu#menu GtkMenuItem#menuitem.separator");
 
     // Label
     case ui::NativeTheme::kColorId_LabelEnabledColor: