MenuButton: support Mac look & feel

ui/views/controls/button/menu_button.h

Index: ui/views/controls/button/menu_button.h
diff --git a/ui/views/controls/button/menu_button.h b/ui/views/controls/button/menu_button.h
index 696a94146f3f9868e01e2eca55a00dfab53f5e8e..d7e4e5adb6197013656c9e9fc46b9fa65696bb79 100644
--- a/ui/views/controls/button/menu_button.h
+++ b/ui/views/controls/button/menu_button.h
@@ -54,10 +54,6 @@ class VIEWS_EXPORT MenuButton : public LabelButton {
   ~MenuButton() override;
 
   bool show_menu_marker() const { return show_menu_marker_; }
-  void set_menu_marker(const gfx::ImageSkia* menu_marker) {
-    menu_marker_ = menu_marker;
-  }
-  const gfx::ImageSkia* menu_marker() const { return menu_marker_; }
 
   const gfx::Point& menu_offset() const { return menu_offset_; }
   void set_menu_offset(int x, int y) { menu_offset_.SetPoint(x, y); }
@@ -130,9 +126,6 @@ class VIEWS_EXPORT MenuButton : public LabelButton {
   // Whether or not we're showing a drop marker.
   bool show_menu_marker_;

[tapted, 2016/04/21 06:06:46]

can this be declared const? that might make it easier to reason about the rest
of the code

[Elly Fong-Jones, 2016/04/21 15:30:26]

Done.

 
-  // The down arrow used to differentiate the menu button from normal buttons.
-  const gfx::ImageSkia* menu_marker_;
-
   // If non-null the destructor sets this to true. This is set while the menu is
   // showing and used to detect if the menu was deleted while running.
   bool* destroyed_flag_;