Hide hover effect on hidden ink drop host views.

ui/views/controls/button/menu_button.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/views/controls/button/menu_button.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "ui/accessibility/ax_view_state.h"
 #include "ui/base/dragdrop/drag_drop_types.h"
 #include "ui/base/l10n/l10n_util.h"
@@ skipping 382 matching lines @@
   // If this was the last lock, manually reset state to the desired state.
   if (pressed_lock_count_ == 0) {
     ButtonState desired_state = STATE_NORMAL;
     if (should_disable_after_press_) {
       desired_state = STATE_DISABLED;
       should_disable_after_press_ = false;
     } else if (ShouldEnterHoveredState()) {
       desired_state = STATE_HOVERED;
     }
     SetState(desired_state);
-    if (ink_drop_delegate() && state() != STATE_PRESSED)
+    // The widget may be null during shutdown.

[bruthig, 2016/06/03 22:35:25]

nit: Can you beef up this comment a little.  It is unclear to me why the call to
OnAction() requires a Widget...

[Evan Stade, 2016/06/03 22:42:40]

Done. Creating an ink drop requires a native theme which we won't have during
shutdown.

+    if (GetWidget() && ink_drop_delegate() && state() != STATE_PRESSED)
       ink_drop_delegate()->OnAction(InkDropState::DEACTIVATED);
   }
 }
 
 int MenuButton::GetMaximumScreenXCoordinate() {
   if (!GetWidget()) {
     NOTREACHED();
     return 0;
   }
 
   gfx::Rect monitor_bounds = GetWidget()->GetWorkAreaBoundsInScreen();
   return monitor_bounds.right() - 1;
 }
 
 }  // namespace views