Enhanced the InkDropRippleImpl to create/destroy InkDropAnimations as needed.

ui/views/animation/ink_drop_state.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_STATE_H_
 #define UI_VIEWS_ANIMATION_INK_DROP_STATE_H_
 
+#include <iosfwd>
+
+#include "ui/views/views_export.h"
+
 namespace views {
 
 // The different states that the ink drop animation can be animated to.
 enum class InkDropState {
   // The ink drop is not visible.
   HIDDEN,
   // The view is being interacted with but the action to be triggered has not
   // yet been determined.
   ACTION_PENDING,
   // The quick action for the view has been triggered. e.g. A tap gesture to

[varkha, 2015/10/06 20:58:38]

nit: here and below s/. e.g. A tap/, e.g. a tap.

[bruthig, 2015/10/08 21:42:55]

Done.

   // click a button.
   QUICK_ACTION,
   // A view is being interacted with and the pending action will be a 'slow'
   // action. e.g. A long press that is still active before releasing.
   SLOW_ACTION_PENDING,
   // The slow action for the view has been triggered. e.g. A long press release
   // to bring up a menu.
   SLOW_ACTION,
   // An active state for a view that is not currently being interacted with.
   // e.g. A pressed button that is showing a menu.
   ACTIVATED,
   // A previously active state has been toggled to inactive. e.g. A drop down
   // menu is closed.
   DEACTIVATED,
 };
 
+VIEWS_EXPORT std::ostream& operator<<(std::ostream& out, InkDropState state);
+

[sadrul, 2015/10/08 01:06:01]

Same here: is this necessary/useful? We have generally not used these, and
instead went with ToString when doing string output for things (see various
gfx::Point etc. things). Maybe go with that?

[bruthig, 2015/10/08 21:42:55]

Replaced with ToString().

 }  // namespace views
 
 #endif  // UI_VIEWS_ANIMATION_INK_DROP_STATE_H_