Add ink drop ripple to shelf overflow button

ui/views/controls/button/custom_button.cc

Index: ui/views/controls/button/custom_button.cc
diff --git a/ui/views/controls/button/custom_button.cc b/ui/views/controls/button/custom_button.cc
index 85dc33627b1d853aebe03296affa86e9b96f3618..4e76eff3a3049aabdc99902052cd1cad0343dd56 100644
--- a/ui/views/controls/button/custom_button.cc
+++ b/ui/views/controls/button/custom_button.cc
@@ -184,6 +184,8 @@ bool CustomButton::OnMouseDragged(const ui::MouseEvent& event) {
 }
 
 void CustomButton::OnMouseReleased(const ui::MouseEvent& event) {
+  const bool was_pressed = state_ == STATE_PRESSED;
+
   if (state_ != STATE_DISABLED) {
     if (!HitTestPoint(event.location())) {
       SetState(STATE_NORMAL);
@@ -197,7 +199,7 @@ void CustomButton::OnMouseReleased(const ui::MouseEvent& event) {
       }
     }
   }
-  if (notify_action_ == NOTIFY_ON_RELEASE)
+  if (notify_action_ == NOTIFY_ON_RELEASE && was_pressed)

[bruthig, 2016/09/01 16:12:54]

Correct me if I'm wrong but you said that OnClickCanceled() can sometimes be
called when the mouse is released with no corresponding mouse press correct?  If
so, can you just add a comment here stating that.

[mohsen, 2016/09/10 03:03:23]

Not exactly. Probably, what I meant was that OnClickCanceled() can be called
even if mouse pressed did not change the button state to PRESSED (e.g. when the
button is disabled). This can be deduced from the comments above
OnClickCanceled() declaration ("Called when a button gets released without
triggering an action").

     OnClickCanceled(event);
 }