Gtk3: Fix border colors on Gtk3.20

chrome/browser/ui/libgtkui/gtk_ui.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/libgtkui/gtk_ui.h"
 
 #include <math.h>
 #include <pango/pango.h>
 #include <X11/Xcursor/Xcursor.h>
 
@@ skipping 810 matching lines @@
       ui::NativeTheme::kColorId_TextfieldDefaultColor);
   // The NTP header is the color that surrounds the current active
   // thumbnail on the NTP, and acts as the border of the "Recent
   // Links" box. It would be awesome if they were separated so we
   // could use GetBorderColor() for the border around the "Recent
   // Links" section, but matching the frame color is more important.
   colors_[ThemeProperties::COLOR_NTP_HEADER] =
       colors_[ThemeProperties::COLOR_FRAME];
 #else
   SkColor toolbar_color = GetBgColor("GtkToolbar#toolbar");
+  SkColor toolbar_text_color = color_utils::GetReadableColor(
+      GetFgColor("GtkToolbar#toolbar GtkLabel#label"),
+      toolbar_color);
 
-  colors_[ThemeProperties::COLOR_TOOLBAR_BUTTON_ICON] =
-      GetFgColor("GtkToolbar#toolbar GtkLabel#label");
+  colors_[ThemeProperties::COLOR_TOOLBAR_BUTTON_ICON] = toolbar_text_color;
 
   // Tabs use the same background color as the toolbar, so use the
   // toolbar text color as the tab text color.
-  SkColor tab_text_color =
-      GetFgColor("GtkToolbar#toolbar.horizontal GtkLabel#label");
-  colors_[ThemeProperties::COLOR_TAB_TEXT] = tab_text_color;
-  colors_[ThemeProperties::COLOR_BOOKMARK_TEXT] = tab_text_color;
+  colors_[ThemeProperties::COLOR_TAB_TEXT] = toolbar_text_color;
+  colors_[ThemeProperties::COLOR_BOOKMARK_TEXT] = toolbar_text_color;
   colors_[ThemeProperties::COLOR_BACKGROUND_TAB_TEXT] =
-      color_utils::BlendTowardOppositeLuma(tab_text_color, 50);
+      color_utils::BlendTowardOppositeLuma(toolbar_text_color, 50);
 
   inactive_selection_bg_color_ =
       GetBgColor("GtkEntry#entry:backdrop #selection:selected");
   inactive_selection_fg_color_ =
       GetFgColor("GtkEntry#entry:backdrop #selection:selected");
 
   SkColor toolbar_separator_horizontal =
       GetSeparatorColor("GtkToolbar#toolbar GtkSeparator#separator.horizontal");
   SkColor toolbar_separator_vertical =
       GetSeparatorColor("GtkToolbar#toolbar GtkSeparator#separator.vertical");
   colors_[ThemeProperties::COLOR_DETACHED_BOOKMARK_BAR_BACKGROUND] =
       toolbar_color;
   colors_[ThemeProperties::COLOR_BOOKMARK_BAR_INSTRUCTIONS_TEXT] =
-      GetFgColor("GtkToolbar#toolbar GtkLabel#label");
+      toolbar_text_color;
   // Separates the toolbar from the bookmark bar or butter bars.
   colors_[ThemeProperties::COLOR_TOOLBAR_BOTTOM_SEPARATOR] =
       toolbar_separator_horizontal;
   // Separates entries in the downloads bar.
   colors_[ThemeProperties::COLOR_TOOLBAR_VERTICAL_SEPARATOR] =
       toolbar_separator_vertical;
   // Separates the bookmark bar from the web content.
   colors_[ThemeProperties::COLOR_DETACHED_BOOKMARK_BAR_SEPARATOR] =
       toolbar_separator_horizontal;
 
   // These colors represent the border drawn around tabs and between
-  // the tabstrip and toolbar.  It's unclear if the selectors used are
-  // correct, but they seem to give reasonable results.
-  SkColor entry_border =
-      GetBorderColor("#headerbar.header-bar.titlebar GtkEntry#entry");
-  SkColor entry_inactive_border =
-      GetBorderColor("#headerbar.header-bar.titlebar:backdrop GtkEntry#entry");
+  // the tabstrip and toolbar.
+  SkColor header_button_border =
+      GetBorderColor("#headerbar.header-bar.titlebar GtkButton#button");
+  SkColor header_button_inactive_border = GetBorderColor(
+      "#headerbar.header-bar.titlebar:backdrop GtkButton#button");
   // Unlike with toolbars, we always want a border around tabs, so let
   // ThemeService choose the border color if the theme doesn't provide one.
-  if (SkColorGetA(entry_border) && SkColorGetA(entry_inactive_border)) {
-    colors_[ThemeProperties::COLOR_TOOLBAR_TOP_SEPARATOR] = entry_border;
+  if (SkColorGetA(header_button_border) &&
+      SkColorGetA(header_button_inactive_border)) {
+    colors_[ThemeProperties::COLOR_TOOLBAR_TOP_SEPARATOR] =
+        header_button_border;
     colors_[ThemeProperties::COLOR_TOOLBAR_TOP_SEPARATOR_INACTIVE] =
-        entry_inactive_border;
+        header_button_inactive_border;
   }
 
-  colors_[ThemeProperties::COLOR_NTP_BACKGROUND] = GetBgColor("");
-  colors_[ThemeProperties::COLOR_NTP_TEXT] = GetFgColor("");
-  colors_[ThemeProperties::COLOR_NTP_HEADER] = GetBorderColor("GtkEntry#entry");
+  SkColor ntp_bg = GetBgColor("");
+  colors_[ThemeProperties::COLOR_NTP_BACKGROUND] = ntp_bg;
+  colors_[ThemeProperties::COLOR_NTP_TEXT] =
+      color_utils::GetReadableColor(GetFgColor("GtkLabel#label"), ntp_bg);
+  colors_[ThemeProperties::COLOR_NTP_HEADER] =
+      GetBorderColor("GtkButton#button");
 #endif
 
   colors_[ThemeProperties::COLOR_TOOLBAR] = toolbar_color;
   colors_[ThemeProperties::COLOR_CONTROL_BACKGROUND] = toolbar_color;
 
   colors_[ThemeProperties::COLOR_NTP_LINK] =
       native_theme_->GetSystemColor(ui::NativeTheme::kColorId_LinkEnabled);
 
   // Generate the colors that we pass to WebKit.
   SetScrollbarColors();
@@ skipping 144 matching lines @@
 
 float GtkUi::GetDeviceScaleFactor() const {
   return device_scale_factor_;
 }
 
 }  // namespace libgtkui
 
 views::LinuxUI* BuildGtkUi() {
   return new libgtkui::GtkUi;
 }