Native Themes: Add table header colors

chrome/browser/ui/libgtkui/native_theme_gtk2.cc

 // Copyright (c) 2013 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/native_theme_gtk2.h"
 
 #include <gtk/gtk.h>
 
 #include "chrome/browser/ui/libgtkui/chrome_gtk_frame.h"
 #include "chrome/browser/ui/libgtkui/chrome_gtk_menu_subclasses.h"
@@ skipping 250 matching lines @@
     case kColorId_TableSelectionBackgroundFocused:
     case kColorId_TableSelectionBackgroundUnfocused:
     case kColorId_TreeSelectionBackgroundFocused:
     case kColorId_TreeSelectionBackgroundUnfocused:
       return GetBgColor(GetTree(), SELECTED);
     case kColorId_TreeArrow:
       return GetFgColor(GetTree(), NORMAL);
     case kColorId_TableGroupingIndicatorColor:
       return GetTextAAColor(GetTree(), NORMAL);
 
+    // Table Headers
+    case kColorId_TableHeaderText:
+      return GetTextColor(GetTree(), NORMAL);
+    case kColorId_TableHeaderBackground:
+      return GetBgColor(GetWindow(), NORMAL);

[Evan Stade, 2017/01/20 01:46:02]

why is this not GetTree?

[Tom (Use chromium acct), 2017/01/20 19:10:38]

The window color is usually slightly darker than the tree color, which would
give a contrast to the header.

But on second thought the tree color is probably the way to go just in case the
theme chooses to use dark trees and light windows.

Done.

+    case kColorId_TableHeaderSeparator:
+      return GetFgColor(GetSeparator(), INSENSITIVE);
+
     // Results Table
     case kColorId_ResultsTableNormalBackground:
       return GetSystemColor(kColorId_TextfieldDefaultBackground);
     case kColorId_ResultsTableHoveredBackground:
       return color_utils::AlphaBlend(
           GetSystemColor(kColorId_TextfieldDefaultBackground),
           GetSystemColor(kColorId_TextfieldSelectionBackgroundFocused), 0x80);
     case kColorId_ResultsTableSelectedBackground:
       return GetSystemColor(kColorId_TextfieldSelectionBackgroundFocused);
     case kColorId_ResultsTableNormalText:
@@ skipping 173 matching lines @@
 GtkWidget* NativeThemeGtk2::GetSeparator() const {
   static GtkWidget* fake_separator = NULL;
 
   if (!fake_separator)
     fake_separator = gtk_hseparator_new();
 
   return fake_separator;
 }
 
 }  // namespace libgtkui