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

Unified Diff: ash/common/system/tray/actionable_view.h

Issue 2499453002: Add ink drop ripple to overview mode button (Closed)
Patch Set: Rebased Created 4 years, 1 month 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 | « ash/common/system/overview/overview_button_tray.cc ('k') | ash/common/system/tray/actionable_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/common/system/tray/actionable_view.h
diff --git a/ash/common/system/tray/actionable_view.h b/ash/common/system/tray/actionable_view.h
index 8d653c4379563587ff1e11d911caa5a96e385f49..555dfb88a75d447c4cc0b8ae84f3110cb8a5eea3 100644
--- a/ash/common/system/tray/actionable_view.h
+++ b/ash/common/system/tray/actionable_view.h
@@ -52,6 +52,14 @@ class ASH_EXPORT ActionableView : public views::ButtonListener,
// been handled and an action was performed. Returns false otherwise.
virtual bool PerformAction(const ui::Event& event) = 0;
+ // Called after PerformAction() to act upon its result, including showing
+ // appropriate ink drop ripple. This will not get called if the view is
+ // destroyed during PerformAction(). Default implementation shows triggered
+ // ripple if action is performed or hides existing ripple if no action is
+ // performed. Subclasses can override to change the default behavior.
+ virtual void HandlePerformActionResult(bool action_performed,
+ const ui::Event& event);
+
// Overridden from views::CustomButton.
const char* GetClassName() const override;
bool OnKeyPressed(const ui::KeyEvent& event) override;
« no previous file with comments | « ash/common/system/overview/overview_button_tray.cc ('k') | ash/common/system/tray/actionable_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698