Gtk3: Use Gtk window bg color as Chrome toolbar color

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 <X11/Xcursor/Xcursor.h>
 #include <dlfcn.h>
 #include <math.h>
 #include <pango/pango.h>
@@ skipping 339 matching lines @@
 }
 
 #if GTK_MAJOR_VERSION > 2
 // COLOR_TOOLBAR_TOP_SEPARATOR represents the border between tabs and the
 // frame, as well as the border between tabs and the toolbar.  For this
 // reason, it is difficult to calculate the One True Color that works well on
 // all themes and is opaque.  However, we can cheat to get a good color that
 // works well for both borders.  The idea is we have two variables: alpha and
 // lightness.  And we have two constraints (on lightness):
 // 1. the border color, when painted on |header_bg|, should give |header_fg|
-// 2. the border color, when painted on |toolbar_bg|, should give |toolbar_fg|
+// 2. the border color, when painted on |tab_bg|, should give |tab_fg|
 // This gives the equations:
 // alpha*lightness + (1 - alpha)*header_bg = header_fg
-// alpha*lightness + (1 - alpha)*toolbar_bg = toolbar_fg
+// alpha*lightness + (1 - alpha)*tab_bg = tab_fg
 // The algorithm below is just a result of solving those equations for alpha
 // and lightness.  If a problem is encountered, like division by zero, or
-// |a| or |l| not in [0, 1], then fallback on |header_fg| or |toolbar_fg|.
+// |a| or |l| not in [0, 1], then fallback on |header_fg| or |tab_fg|.
 SkColor GetToolbarTopSeparatorColor(SkColor header_fg,
                                     SkColor header_bg,
-                                    SkColor toolbar_fg,
-                                    SkColor toolbar_bg) {
+                                    SkColor tab_fg,
+                                    SkColor tab_bg) {
   using namespace color_utils;
 
-  SkColor default_color = SkColorGetA(header_fg) ? header_fg : toolbar_fg;
+  SkColor default_color = SkColorGetA(header_fg) ? header_fg : tab_fg;
   if (!SkColorGetA(default_color))
     return SK_ColorTRANSPARENT;
 
   auto get_lightness = [](SkColor color) {
     HSL hsl;
     SkColorToHSL(color, &hsl);
     return hsl.l;
   };
 
   double f1 = get_lightness(GetResultingPaintColor(header_fg, header_bg));
   double b1 = get_lightness(header_bg);
-  double f2 = get_lightness(GetResultingPaintColor(toolbar_fg, toolbar_bg));
-  double b2 = get_lightness(toolbar_bg);
+  double f2 = get_lightness(GetResultingPaintColor(tab_fg, tab_bg));
+  double b2 = get_lightness(tab_bg);
 
   if (b1 == b2)
     return default_color;
   double a = (f1 - f2 - b1 + b2) / (b2 - b1);
   if (a == 0)
     return default_color;
   double l = (f1 - (1 - a) * b1) / a;
   if (a < 0 || a > 1 || l < 0 || l > 1)
     return default_color;
   // Take the hue and saturation from |default_color|, but use the
@@ skipping 423 matching lines @@
 
   temp_color = color_utils::HSLShift(frame_color, kDefaultTintFrameIncognito);
   GetChromeStyleColor("incognito-frame-color", &temp_color);
   colors_[ThemeProperties::COLOR_FRAME_INCOGNITO] = temp_color;
 
   temp_color =
       color_utils::HSLShift(frame_color, kDefaultTintFrameIncognitoInactive);
   GetChromeStyleColor("incognito-inactive-frame-color", &temp_color);
   colors_[ThemeProperties::COLOR_FRAME_INCOGNITO_INACTIVE] = temp_color;
 
-  SkColor toolbar_color =
+  SkColor tab_color =
       native_theme_->GetSystemColor(ui::NativeTheme::kColorId_DialogBackground);
   SkColor label_color = native_theme_->GetSystemColor(
       ui::NativeTheme::kColorId_LabelEnabledColor);
 
   colors_[ThemeProperties::COLOR_TOOLBAR_BUTTON_ICON] =
       color_utils::DeriveDefaultIconColor(label_color);
 
   colors_[ThemeProperties::COLOR_TAB_TEXT] = label_color;
   colors_[ThemeProperties::COLOR_BOOKMARK_TEXT] = label_color;
   colors_[ThemeProperties::COLOR_BACKGROUND_TAB_TEXT] =
       color_utils::BlendTowardOppositeLuma(label_color, 50);
 
   inactive_selection_bg_color_ = native_theme_->GetSystemColor(
       ui::NativeTheme::kColorId_TextfieldReadOnlyBackground);
   inactive_selection_fg_color_ = native_theme_->GetSystemColor(
       ui::NativeTheme::kColorId_TextfieldReadOnlyColor);
 
   // We pick the text and background colors for the NTP out of the
   // colors for a GtkEntry. We do this because GtkEntries background
-  // color is never the same as |toolbar_color|, is usually a white,
+  // color is never the same as |tab_color|, is usually a white,
   // and when it isn't a white, provides sufficient contrast to
-  // |toolbar_color|. Try this out with Darklooks, HighContrastInverse
+  // |tab_color|. Try this out with Darklooks, HighContrastInverse
   // or ThinIce.
   colors_[ThemeProperties::COLOR_NTP_BACKGROUND] =
       native_theme_->GetSystemColor(
           ui::NativeTheme::kColorId_TextfieldDefaultBackground);
   colors_[ThemeProperties::COLOR_NTP_TEXT] = native_theme_->GetSystemColor(
       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
   std::string header_selector = GtkVersionCheck(3, 10)
                                     ? "#headerbar.header-bar.titlebar"
                                     : "GtkMenuBar#menubar";
   SkColor frame_color = GetBgColor(header_selector);
   SkColor frame_color_inactive = GetBgColor(header_selector + ":backdrop");
   colors_[ThemeProperties::COLOR_FRAME] = frame_color;
   colors_[ThemeProperties::COLOR_FRAME_INACTIVE] = frame_color_inactive;
   colors_[ThemeProperties::COLOR_FRAME_INCOGNITO] =
       color_utils::HSLShift(frame_color, kDefaultTintFrameIncognito);
   colors_[ThemeProperties::COLOR_FRAME_INCOGNITO_INACTIVE] =
       color_utils::HSLShift(frame_color_inactive, kDefaultTintFrameIncognito);
 
-  SkColor toolbar_color = GetBgColor("GtkToolbar#toolbar");
-  SkColor toolbar_text_color = color_utils::GetReadableColor(
-      GetFgColor("GtkToolbar#toolbar GtkLabel#label"),
-      toolbar_color);
+  SkColor tab_color = GetBgColor("");

[Elliot Glaysher, 2017/03/30 20:34:19]

Slightly surprised that this can be empty; I would have expected you to have to
specify GtkWindow, but I guess that's part of the style context by default?

[Tom (Use chromium acct), 2017/03/30 20:40:26]

https://cs.chromium.org/chromium/src/chrome/browser/ui/libgtkui/gtk_util.cc?r...

+  SkColor tab_text_color = GetFgColor("GtkLabel");
 
-  colors_[ThemeProperties::COLOR_TOOLBAR_BUTTON_ICON] = toolbar_text_color;
+  colors_[ThemeProperties::COLOR_TOOLBAR_BUTTON_ICON] = tab_text_color;
 
-  // Tabs use the same background color as the toolbar, so use the
-  // toolbar text color as the tab text color.
-  colors_[ThemeProperties::COLOR_TAB_TEXT] = toolbar_text_color;
-  colors_[ThemeProperties::COLOR_BOOKMARK_TEXT] = toolbar_text_color;
+  colors_[ThemeProperties::COLOR_TAB_TEXT] = tab_text_color;
+  colors_[ThemeProperties::COLOR_BOOKMARK_TEXT] = tab_text_color;
   colors_[ThemeProperties::COLOR_BACKGROUND_TAB_TEXT] =
-      color_utils::BlendTowardOppositeLuma(toolbar_text_color, 50);
+      color_utils::BlendTowardOppositeLuma(tab_text_color, 50);
 
-  SkColor location_bar_border =
-      GetBorderColor("GtkToolbar#toolbar GtkEntry#entry");
+  SkColor location_bar_border = GetBorderColor("GtkEntry#entry");
   if (SkColorGetA(location_bar_border))
     colors_[ThemeProperties::COLOR_LOCATION_BAR_BORDER] = location_bar_border;
 
   inactive_selection_bg_color_ = GetSelectionBgColor(
       GtkVersionCheck(3, 20) ? "GtkTextView#textview.view:backdrop "
                                "#text:backdrop #selection:backdrop"
                              : "GtkTextView.view:selected:backdrop");
   inactive_selection_fg_color_ =
       GetFgColor(GtkVersionCheck(3, 20) ? "GtkTextView#textview.view:backdrop "
                                           "#text:backdrop #selection:backdrop"
                                         : "GtkTextView.view:selected:backdrop");
 
-  SkColor toolbar_button_border =
-      GetBorderColor("GtkToolbar#toolbar GtkButton#button");
-  colors_[ThemeProperties::COLOR_DETACHED_BOOKMARK_BAR_BACKGROUND] =
-      toolbar_color;
+  SkColor tab_border = GetBorderColor("GtkButton#button");
+  colors_[ThemeProperties::COLOR_DETACHED_BOOKMARK_BAR_BACKGROUND] = tab_color;
   colors_[ThemeProperties::COLOR_BOOKMARK_BAR_INSTRUCTIONS_TEXT] =
-      toolbar_text_color;
+      tab_text_color;
   // Separates the toolbar from the bookmark bar or butter bars.
-  colors_[ThemeProperties::COLOR_TOOLBAR_BOTTOM_SEPARATOR] =
-      toolbar_button_border;
+  colors_[ThemeProperties::COLOR_TOOLBAR_BOTTOM_SEPARATOR] = tab_border;
   // Separates entries in the downloads bar.
-  colors_[ThemeProperties::COLOR_TOOLBAR_VERTICAL_SEPARATOR] =
-      toolbar_button_border;
+  colors_[ThemeProperties::COLOR_TOOLBAR_VERTICAL_SEPARATOR] = tab_border;
   // Separates the bookmark bar from the web content.
-  colors_[ThemeProperties::COLOR_DETACHED_BOOKMARK_BAR_SEPARATOR] =
-      toolbar_button_border;
+  colors_[ThemeProperties::COLOR_DETACHED_BOOKMARK_BAR_SEPARATOR] = tab_border;
 
   // These colors represent the border drawn around tabs and between
   // the tabstrip and toolbar.
   SkColor toolbar_top_separator = GetToolbarTopSeparatorColor(
       GetBorderColor(header_selector + " GtkButton#button"), frame_color,
-      toolbar_button_border, toolbar_color);
+      tab_border, tab_color);
   SkColor toolbar_top_separator_inactive = GetToolbarTopSeparatorColor(
       GetBorderColor(header_selector + ":backdrop GtkButton#button"),
-      frame_color_inactive, toolbar_button_border, toolbar_color);
+      frame_color_inactive, tab_border, tab_color);
   // 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(toolbar_top_separator) &&
       SkColorGetA(toolbar_top_separator_inactive)) {
     colors_[ThemeProperties::COLOR_TOOLBAR_TOP_SEPARATOR] =
         toolbar_top_separator;
     colors_[ThemeProperties::COLOR_TOOLBAR_TOP_SEPARATOR_INACTIVE] =
         toolbar_top_separator_inactive;
   }
 
   colors_[ThemeProperties::COLOR_NTP_BACKGROUND] =
       native_theme_->GetSystemColor(
           ui::NativeTheme::kColorId_TextfieldDefaultBackground);
   colors_[ThemeProperties::COLOR_NTP_TEXT] = native_theme_->GetSystemColor(
       ui::NativeTheme::kColorId_TextfieldDefaultColor);
   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_TOOLBAR] = tab_color;
+  colors_[ThemeProperties::COLOR_CONTROL_BACKGROUND] = tab_color;
 
   colors_[ThemeProperties::COLOR_NTP_LINK] = native_theme_->GetSystemColor(
       ui::NativeTheme::kColorId_TextfieldSelectionBackgroundFocused);
 
   // Generate the colors that we pass to WebKit.
   SetScrollbarColors();
   focus_ring_color_ = native_theme_->GetSystemColor(
       ui::NativeTheme::kColorId_FocusedBorderColor);
 
   // Some GTK themes only define the text selection colors on the GtkEntry
   // class, so we need to use that for getting selection colors.
   active_selection_bg_color_ = native_theme_->GetSystemColor(
       ui::NativeTheme::kColorId_TextfieldSelectionBackgroundFocused);
   active_selection_fg_color_ = native_theme_->GetSystemColor(
       ui::NativeTheme::kColorId_TextfieldSelectionColor);
 
-  SkColor throbber_spinning = native_theme_->GetSystemColor(
-      ui::NativeTheme::kColorId_ThrobberSpinningColor);
   colors_[ThemeProperties::COLOR_TAB_THROBBER_SPINNING] =
-      color_utils::GetReadableColor(throbber_spinning, toolbar_color);
-  SkColor throbber_waiting = native_theme_->GetSystemColor(
-      ui::NativeTheme::kColorId_ThrobberWaitingColor);
+      native_theme_->GetSystemColor(
+          ui::NativeTheme::kColorId_ThrobberSpinningColor);
   colors_[ThemeProperties::COLOR_TAB_THROBBER_WAITING] =
-      color_utils::GetReadableColor(throbber_waiting, toolbar_color);
+      native_theme_->GetSystemColor(
+          ui::NativeTheme::kColorId_ThrobberWaitingColor);
 }
 
 void GtkUi::UpdateCursorTheme() {
   GtkSettings* settings = gtk_settings_get_default();
 
   gchar* theme = nullptr;
   gint size = 0;
   g_object_get(settings, "gtk-cursor-theme-name", &theme,
                "gtk-cursor-theme-size", &size, nullptr);
 
@@ skipping 84 matching lines @@
 
 float GtkUi::GetDeviceScaleFactor() const {
   return device_scale_factor_;
 }
 
 }  // namespace libgtkui
 
 views::LinuxUI* BuildGtkUi() {
   return new libgtkui::GtkUi;
 }