[AiS] for desktop, two lines and font sytles

chrome/browser/ui/libgtk2ui/native_theme_gtk2.cc

Index: chrome/browser/ui/libgtk2ui/native_theme_gtk2.cc
diff --git a/chrome/browser/ui/libgtk2ui/native_theme_gtk2.cc b/chrome/browser/ui/libgtk2ui/native_theme_gtk2.cc
index 3dec05e03c2df7ed5ca6e361999b24e5561e8570..6030ab8bdf654a80958d927915571c91851b828b 100644
--- a/chrome/browser/ui/libgtk2ui/native_theme_gtk2.cc
+++ b/chrome/browser/ui/libgtk2ui/native_theme_gtk2.cc
@@ -373,6 +373,40 @@ GdkColor NativeThemeGtk2::GetSystemGdkColor(ColorId color_id) const {
       return GdkAlphaBlend(win_style->text[GTK_STATE_SELECTED],
                            win_style->bg[GTK_STATE_SELECTED], 0x34);
     }
+    case kColorId_ResultsTablePositiveText: {
+      GdkColor color = {0};
+      color.green = ~0;
+      return color;

[Peter Kasting, 2015/03/14 01:54:13]

You should use GetReadableColor() anytime you hardcode a source color, to ensure
it's readable over the background.  See how native_theme_win.cc does this.

[dschuyler, 2015/03/16 21:49:09]

Done.

+    }
+    case kColorId_ResultsTablePositiveHoveredText: {
+      GdkColor color = {0};
+      color.green = ~0;
+      GtkStyle* entry_style = GetEntryStyle();
+      return GdkAlphaBlend(color, entry_style->base[GTK_STATE_PRELIGHT], 0x80);
+    }
+    case kColorId_ResultsTablePositiveSelectedText: {
+      GdkColor color = {0};
+      color.green = ~0;
+      GtkStyle* entry_style = GetEntryStyle();
+      return GdkAlphaBlend(color, entry_style->base[GTK_STATE_SELECTED], 0x80);

[Peter Kasting, 2015/03/14 01:54:14]

I don't think this is what you want.  You're making the selected text much lower
contrast by blending it with the selected background.  We use that trick above
to dim text, not to adjust for being selected.

I think instead what you want to do is use GetReadableColor() again as suggested
above, but with the selected background color instead of the normal one.

[dschuyler, 2015/03/16 21:49:09]

Done.

+    }
+    case kColorId_ResultsTableNegativeText: {
+      GdkColor color = {0};
+      color.red = ~0;
+      return color;
+    }
+    case kColorId_ResultsTableNegativeHoveredText: {
+      GdkColor color = {0};
+      color.red = ~0;
+      GtkStyle* entry_style = GetEntryStyle();
+      return GdkAlphaBlend(color, entry_style->base[GTK_STATE_PRELIGHT], 0x80);
+    }
+    case kColorId_ResultsTableNegativeSelectedText: {
+      GdkColor color = {0};
+      color.red = ~0;
+      GtkStyle* entry_style = GetEntryStyle();
+      return GdkAlphaBlend(color, entry_style->base[GTK_STATE_SELECTED], 0x80);
+    }
     case kColorId_NumColors:
       NOTREACHED();
       break;