Show/hide ink drop highlight when button enable state changes.

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 21ebca3d538420116b4a012c0720d365c4a5612d..115dcb4b486be897aa89d77250af8ee40d0c7e31 100644
--- a/ui/views/controls/button/custom_button.cc
+++ b/ui/views/controls/button/custom_button.cc
@@ -139,6 +139,7 @@ void CustomButton::OnEnabledChanged() {
     GetInkDrop()->SetHovered(should_enter_hover_state);
   } else {
     SetState(STATE_DISABLED);
+    GetInkDrop()->SetHovered(false);

[sadrul, 2017/01/03 17:13:25]

Can some of these move into CustomButton::StateChanged()?

[bruthig, 2017/01/03 20:05:21]

Would it make more sense to move this to InkDropHostView::OnEnabledChanged() ?

[Evan Stade, 2017/01/03 21:19:38]

I don't know, maybe, but I'd rather keep it near the parallel branch (i.e. the
enabled case). I guess it might be nice if both of them moved to InkDropHostView
but that would require a virtual ShouldEnterHoveredState or something.

[Evan Stade, 2017/01/03 21:19:38]

Not easily. For one thing, all the 19 overrides would suddenly have to call
through to the parent OnStateChanged. I think it's unfortunate/undesirable to
have these two separate, parallel versions of button state, and it's on Ben's
radar but non trivial to fix.

   }
 }