Adds ripple animation to extension buttons on a toolbar

ui/views/controls/button/menu_button.cc

Index: ui/views/controls/button/menu_button.cc
diff --git a/ui/views/controls/button/menu_button.cc b/ui/views/controls/button/menu_button.cc
index 5603548ba337ab554a5e288b4a3ec353172c7297..1225e67f5057482bc8db59bf65543a12a7f14603 100644
--- a/ui/views/controls/button/menu_button.cc
+++ b/ui/views/controls/button/menu_button.cc
@@ -145,6 +145,12 @@ bool MenuButton::Activate() {
   return true;
 }
 
+void MenuButton::WillNotActivate() {
+  if (listener_) {
+    listener_->OnMenuButtonClickCanceled(this);

[sadrul, 2015/11/18 22:56:31]

no {} for this

[varkha, 2015/11/18 23:27:54]

This is inlined in OnMouseReleased (see
https://codereview.chromium.org/1411833006/).

+  }
+}
+
 void MenuButton::OnPaint(gfx::Canvas* canvas) {
   LabelButton::OnPaint(canvas);
 
@@ -189,6 +195,7 @@ void MenuButton::OnMouseReleased(const ui::MouseEvent& event) {
       HitTestPoint(event.location()) && !InDrag()) {
     Activate();
   } else {
+    WillNotActivate();

[sadrul, 2015/11/18 22:56:31]

Can this directly call into listener_->OnMenuButtonClickCanceled()?

[varkha, 2015/11/18 23:27:54]

Done in https://codereview.chromium.org/1411833006/.

     LabelButton::OnMouseReleased(event);
   }
 }