wrench_menu: use NativeTheme colors under aura

chrome/browser/ui/views/wrench_menu.cc

Index: chrome/browser/ui/views/wrench_menu.cc
diff --git a/chrome/browser/ui/views/wrench_menu.cc b/chrome/browser/ui/views/wrench_menu.cc
index fce9d4348c6c127a10f190178fec6c43ce386a25..9a0a121aecaccae72dc552ae22c25ca4488d9b17 100644
--- a/chrome/browser/ui/views/wrench_menu.cc
+++ b/chrome/browser/ui/views/wrench_menu.cc
@@ -248,6 +248,37 @@ class MenuButtonBackground : public views::Background {
   }
 
  private:
+#if defined(USE_AURA)

[Elliot Glaysher, 2013/03/18 17:53:40]

Please remember that aura is not a linux-only component, but is currently
shipped on chromeos and will be shipped on Windows, probably simultaneously with
linux.

Querying a NativeTheme (preferably via Views::GetNativeTheme()) is fine, but
please push the current colors up into all the subclasses of NativeTheme, and
override the colors returned only in NativeThemeGtk2. (This is something that's
been on various people's plates for a while, but is now actually a thing that
has to happen for linux_aura.)

[Bobby Powers, 2013/03/18 18:11:43]

{Unfocused,Focused,Menu}BorderColor were already defined for NativeThemeAura,
the only new color I added was HotMenuItemBackgroundColor.  I defined this for
NativeThemeAura (using the original color from wrench_menu.cc), and overrode it
in NativeThemeGtk2.  It seems like the best way to make sure it is available to
Windows & friends is to add it to CommonThemeGetSystemColor.  Does this make
sense, or should I explicitly override it NativeThemeWin?

[Bobby Powers, 2013/03/18 19:59:53]

I believe the current patch addresses this correctly.

+  static SkColor border_color(CustomButton::ButtonState state) {
+    ui::NativeTheme *theme = ui::NativeTheme::instance();
+    switch (state) {
+      case CustomButton::STATE_HOVERED:
+        return theme->GetSystemColor(
+          ui::NativeTheme::kColorId_UnfocusedBorderColor);
+      case CustomButton::STATE_PRESSED:
+        return theme->GetSystemColor(
+          ui::NativeTheme::kColorId_FocusedBorderColor);
+      default:
+        return theme->GetSystemColor(
+          ui::NativeTheme::kColorId_MenuBorderColor);
+    }
+  }
+
+  static SkColor background_color(CustomButton::ButtonState state) {
+    ui::NativeTheme *theme = ui::NativeTheme::instance();
+    switch (state) {
+      case CustomButton::STATE_HOVERED:
+        return theme->GetSystemColor(
+          ui::NativeTheme::kColorId_HotMenuItemBackgroundColor);
+      case CustomButton::STATE_PRESSED:
+        return theme->GetSystemColor(
+          ui::NativeTheme::kColorId_FocusedMenuItemBackgroundColor);
+      default:
+        return theme->GetSystemColor(
+          ui::NativeTheme::kColorId_MenuBackgroundColor);
+    }
+  }
+#else
   static SkColor border_color(CustomButton::ButtonState state) {
     switch (state) {
       case CustomButton::STATE_HOVERED: return kHotBorderColor;
@@ -263,6 +294,7 @@ class MenuButtonBackground : public views::Background {
       default:                          return kBackgroundColor;
     }
   }
+#endif
 
   static SkColor touch_background_color(CustomButton::ButtonState state) {
     switch (state) {