Set bookmarks bar text color to #646464 in MD mode

chrome/browser/themes/theme_properties.cc

Index: chrome/browser/themes/theme_properties.cc
diff --git a/chrome/browser/themes/theme_properties.cc b/chrome/browser/themes/theme_properties.cc
index 770b3dd61b00654857a6631e6305a160e3272707..a03a4715b5491808f49d05b3e91eae81a1c4f248 100644
--- a/chrome/browser/themes/theme_properties.cc
+++ b/chrome/browser/themes/theme_properties.cc
@@ -88,9 +88,10 @@ const SkColor kDefaultColorBackgroundTabTextIncognito[] = {
     SkColorSetRGB(0x40, 0x40, 0x40), SK_ColorWHITE};
 #endif  // OS_MACOSX
 
-constexpr SkColor kDefaultColorBookmarkText = SK_ColorBLACK;
-constexpr SkColor kDefaultColorBookmarkTextIncognito[] = {
-    kDefaultColorBookmarkText, SK_ColorWHITE};
+const SkColor kDefaultColorBookmarkText[] = {
+    SK_ColorBLACK, SkColorSetRGB(0x64, 0x64, 0x64)};

[Evan Stade, 2016/06/14 14:40:30]

won't changing this also impact the attached bar as well as the bookmark
buttons? The bug says "empty bookmarks bar" but doesn't specify attached vs
detached. The bg for each of these cases is different so I suspect the fg should
be different too.

The bug also doesn't say the bookmark button text should change. 

[kylix_rd, 2016/06/14 14:47:22]

I've uploaded images to the bug report for comment. To me, the text remains
readable in either mode with the same foreground color.

+const SkColor kDefaultColorBookmarkTextIncognito[] = {
+    SK_ColorBLACK, SkColorSetARGB(0x8A, 0xFF, 0xFF, 0xFF)};
 
 #if defined(OS_WIN)
 const SkColor kDefaultColorNTPBackground =
@@ -333,7 +334,7 @@ SkColor ThemeProperties::GetDefaultColor(int id, bool otr) {
                  : kDefaultColorBackgroundTabText[mode];
     case COLOR_BOOKMARK_TEXT:
       return otr ? kDefaultColorBookmarkTextIncognito[mode]
-                 : kDefaultColorBookmarkText;
+                 : kDefaultColorBookmarkText[mode];
     case COLOR_NTP_BACKGROUND:
       return kDefaultColorNTPBackground;
     case COLOR_NTP_TEXT: