Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(755)

Unified Diff: ui/views/controls/button/menu_button.cc

Issue 1757993004: Added ink drop hover/ripple to menu hosting bookmark buttons. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed sky@ & varkha@ comments. Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ui/views/controls/button/menu_button.h ('k') | ui/views/controls/menu/menu_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 599e06a72ff06d91f657d5747a94636da70c4c87..41921b923dbabc7f804ee752d0ffa7a353b0eb50 100644
--- a/ui/views/controls/button/menu_button.cc
+++ b/ui/views/controls/button/menu_button.cc
@@ -47,8 +47,12 @@ const int MenuButton::kMenuMarkerPaddingRight = -1;
////////////////////////////////////////////////////////////////////////////////
MenuButton::PressedLock::PressedLock(MenuButton* menu_button)
+ : PressedLock(menu_button, false) {}
+
+MenuButton::PressedLock::PressedLock(MenuButton* menu_button,
+ bool is_sibling_menu_show)
: menu_button_(menu_button->weak_factory_.GetWeakPtr()) {
- menu_button_->IncrementPressedLocked();
+ menu_button_->IncrementPressedLocked(is_sibling_menu_show);
}
MenuButton::PressedLock::~PressedLock() {
@@ -368,13 +372,17 @@ void MenuButton::NotifyClick(const ui::Event& event) {
Activate(&event);
}
-void MenuButton::IncrementPressedLocked() {
+void MenuButton::IncrementPressedLocked(bool snap_ink_drop_to_activated) {
++pressed_lock_count_;
if (increment_pressed_lock_called_)
*increment_pressed_lock_called_ = true;
should_disable_after_press_ = state() == STATE_DISABLED;
- if (state() != STATE_PRESSED && ink_drop_delegate())
- ink_drop_delegate()->OnAction(InkDropState::ACTIVATED);
+ if (state() != STATE_PRESSED && ink_drop_delegate()) {
+ if (snap_ink_drop_to_activated)
+ ink_drop_delegate()->SnapToActivated();
+ else
+ ink_drop_delegate()->OnAction(InkDropState::ACTIVATED);
+ }
SetState(STATE_PRESSED);
}
« no previous file with comments | « ui/views/controls/button/menu_button.h ('k') | ui/views/controls/menu/menu_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698