Added ink drop hover/ripple to menu hosting bookmark buttons.

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 91504d2e795d1d109ad177732c63946a32c4ca5b..01bbb132a10285b7f08ba3286afc29fa6ec500e2 100644
--- a/ui/views/controls/button/menu_button.h
+++ b/ui/views/controls/button/menu_button.h
@@ -32,6 +32,7 @@ class VIEWS_EXPORT MenuButton : public LabelButton {
   class VIEWS_EXPORT PressedLock {
    public:
     explicit PressedLock(MenuButton* menu_button);
+    PressedLock(MenuButton* menu_button, bool first_menu_show);

[sky, 2016/03/11 20:41:32]

Rather than explicitly supplying the state can you determine it from whether
there is a MenuController around?

[bruthig, 2016/03/11 21:58:20]

Naively, I'm not aware of any way for a MenuButton to determine if a
MenuController is around and/or active.  But even if it were possible it depends
more on whether the button is becoming pressed due to the MenuController::Run()
method or the MenuController::ShowSiblingMenu(). AFAICT the
MenuButton::PressedLock is the glue between the MenuController and the
MenuButton.

[sky, 2016/03/11 23:11:14]

MenuController::GetActiveInstance, but you're right on the sibling part. Ok.

     ~PressedLock();
 
    private:
@@ -105,8 +106,10 @@ class VIEWS_EXPORT MenuButton : public LabelButton {
   friend class PressedLock;
 
   // Increment/decrement the number of "pressed" locks this button has, and
-  // set the state accordingly.
-  void IncrementPressedLocked();
+  // set the state accordingly. The ink drop is animated to the ACTIVATED state
+  // if |animate_ink_drop| is true and will be snapped to it's final frame if
+  // false.
+  void IncrementPressedLocked(bool animate_ink_drop);
   void DecrementPressedLocked();
 
   // Compute the maximum X coordinate for the current screen. MenuButtons