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

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

Issue 2499453002: Add ink drop ripple to overview mode button (Closed)
Patch Set: Addressed review comments 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
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 1dd60b4b41b68d3e63f6cf764c9ad286868baf64..7936335c997e2e2865f27cbc07213ae3fad650c3 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;

Powered by Google App Engine
This is Rietveld 408576698