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:
+    case kColorId_TableHeaderBackground:
+    case kColorId_TableHeaderSeparator:
+      // Fallback to using common theme colors.
+      return ui::NativeTheme::GetInstanceForNativeUi()->GetSystemColor(

[Evan Stade, 2017/01/19 00:49:22]

why fall back to common colors? that will give us a light bg instead of a dark
one on dark themes.

[Tom (Use chromium acct), 2017/01/19 02:03:23]

Changed this to get the colors directly

+          color_id);
+
     // 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