Refactoring to make adding ink drop animations easier

ui/views/animation/ink_drop_delegate.h

+// Copyright 2015 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.
+
+#ifndef UI_VIEWS_ANIMATION_INK_DROP_DELEGATE_H_
+#define UI_VIEWS_ANIMATION_INK_DROP_DELEGATE_H_
+
+#include "base/macros.h"
+#include "ui/views/views_export.h"
+
+namespace ui {
+class GestureEvent;
+class Event;
+}  // namespace ui
+
+namespace views {
+
+// Ink ripple animation delegate that starts and stops animations based on
+// View states and events.
+class VIEWS_EXPORT InkDropDelegate {
+ public:
+  InkDropDelegate() {}
+  virtual ~InkDropDelegate() {}
+
+  // Sets sizes for the animation layers.
+  virtual void SetInkDropSize(int large_size,
+                              int large_corner_radius,
+                              int small_size,
+                              int small_corner_radius) = 0;
+
+  // Called when the View's layout changes necessitating change in positioning
+  // of ink ripple layers.
+  virtual void OnLayout() = 0;
+
+  // Called when ink ripple should be hidden.
+  virtual void OnActionComplete() = 0;

[bruthig, 2015/11/16 22:37:11]

What do you think about having a single function OnAction(InkDropState)?  That
way we wouldn't have to update this class if we edit the InkDropState enum and I
think it's fair to say it should be one-to-one.

[varkha, 2015/11/18 22:56:44]

Done.

+
+  // Called to activate the ink ripple.
+  virtual void OnActionPending() = 0;
+
+  // Called to start showing QUICK_ACTION ripple.
+  virtual void OnActionQuick() = 0;
+
+  // Called to start showing ACTIVATED (menu) ripple.
+  virtual void OnActivated() = 0;
+
+  // Called to animate end of ACTIVATED (menu) ripple.
+  virtual void OnDeactivated() = 0;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(InkDropDelegate);
+};
+
+}  // namespace views
+
+#endif  // UI_VIEWS_ANIMATION_INK_DROP_DELEGATE_H_