wrench_menu: use NativeTheme colors under aura

chrome/browser/ui/views/wrench_menu.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_WRENCH_MENU_H_
 #define CHROME_BROWSER_UI_VIEWS_WRENCH_MENU_H_
 
 #include <map>
 #include <utility>
 
@@ skipping 23 matching lines @@
 class WrenchMenu : public views::MenuDelegate,
                    public BaseBookmarkModelObserver,
                    public content::NotificationObserver {
  public:
   // TODO: remove |use_new_menu| and |supports_new_separators|.
   WrenchMenu(Browser* browser,
              bool use_new_menu,
              bool supports_new_separators);
   virtual ~WrenchMenu();
 
-  void Init(ui::MenuModel* model);
+  void Init(views::View* view, ui::MenuModel* model);

[sky, 2013/03/19 21:08:26]

It seems as though the only need you have for view is to get the native theme.
See code in WrenchMenu::GetMenuConfig() for how to get the NativeTheme from the
browser_.

[Bobby Powers, 2013/03/20 14:54:38]

Ah, yes!  Very good.  I've reworked the patch to do exactly that.

 
   // Shows the menu relative to the specified view.
   void RunMenu(views::MenuButton* host);
 
   // Whether the menu is currently visible to the user.
   bool IsShowing();
 
   const views::MenuConfig& GetMenuConfig() const;
 
   bool use_new_menu() const { return use_new_menu_; }
@@ skipping 45 matching lines @@
   class ZoomView;
 
   typedef std::pair<ui::MenuModel*,int> Entry;
   typedef std::map<int,Entry> IDToEntry;
 
   // Populates |parent| with all the child menus in |model|. Recursively invokes
   // |PopulateMenu| for any submenu. |next_id| is incremented for every menu
   // that is created.
   void PopulateMenu(views::MenuItemView* parent,
                     ui::MenuModel* model,
+                    views::View* view,
                     int* next_id);
 
   // Adds a new menu to |parent| to represent the MenuModel/index pair passed
   // in.
   // Fur button containing menu items a |height| override can be specified with
   // a number bigger then 0.
   views::MenuItemView* AppendMenuItem(views::MenuItemView* parent,
                                       ui::MenuModel* model,
                                       int index,
                                       ui::MenuModel::ItemType menu_type,
@@ skipping 50 matching lines @@
   content::NotificationRegistrar registrar_;
 
   const bool use_new_menu_;
 
   const bool supports_new_separators_;
 
   DISALLOW_COPY_AND_ASSIGN(WrenchMenu);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_WRENCH_MENU_H_