[Views] Update ink drop for omnibox icons

ui/views/animation/ink_drop.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_H_
 #define UI_VIEWS_ANIMATION_INK_DROP_H_
 
 #include <memory>
 
 #include "base/macros.h"
 #include "base/time/time.h"
 #include "ui/compositor/layer_tree_owner.h"
 #include "ui/events/event_handler.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/geometry/size.h"
 #include "ui/views/animation/ink_drop_state.h"
 #include "ui/views/view.h"
 #include "ui/views/views_export.h"
 
 namespace views {
 
+class InkDropObserver;
+
 // Pure virtual base class that manages the lifetime and state of an ink drop

[bruthig, 2017/04/21 15:05:10]

Can you update the docs?  This is no longer a pure-virtual class.

[spqchan, 2017/04/27 21:40:06]

Done.

 // ripple as well as visual hover state feedback.
 class VIEWS_EXPORT InkDrop {
  public:
-  virtual ~InkDrop() {}
+  virtual ~InkDrop();
 
   // Called by ink drop hosts when their size is changed.
   virtual void HostSizeChanged(const gfx::Size& new_size) = 0;
 
   // Gets the target state of the ink drop.
   virtual InkDropState GetTargetInkDropState() const = 0;
 
   // Animates from the current InkDropState to |ink_drop_state|.
   virtual void AnimateToState(InkDropState ink_drop_state) = 0;
 
   // Immediately snaps the InkDropState to ACTIVATED. This more specific
   // implementation of the non-existent SnapToState(InkDropState) function is
   // the only one available because it was the only InkDropState that clients
   // needed to skip animations for.
   virtual void SnapToActivated() = 0;
 
   // Enables or disables the hover state.
   virtual void SetHovered(bool is_hovered) = 0;
 
   // Enables or disables the focus state.
   virtual void SetFocused(bool is_focused) = 0;
 
+  // Returns true if the highlight animation is in the process of fading in or
+  // is visible.
+  virtual bool IsHighlightFadingInOrVisible() const = 0;
+
+  // Methods to add/remove observers for this object.
+  void AddObserver(InkDropObserver* observer);
+  void RemoveObserver(InkDropObserver* observer);
+
  protected:
-  InkDrop() {}
+  InkDrop();
+
+  const base::ObserverList<InkDropObserver>* observers() const {
+    return &observers_;
+  }
 
  private:
+  base::ObserverList<InkDropObserver> observers_;
+
   DISALLOW_COPY_AND_ASSIGN(InkDrop);
 };
 
 // A View which can be used to parent ink drop layers. Typically this is used
 // as a non-ancestor view to labels so that the labels can paint on an opaque
 // canvas. This is used to avoid ugly text renderings when labels with subpixel
 // rendering enabled are painted onto a non-opaque canvas.
 class VIEWS_EXPORT InkDropContainerView : public views::View {
  public:
   InkDropContainerView();
 
   void AddInkDropLayer(ui::Layer* ink_drop_layer);
   void RemoveInkDropLayer(ui::Layer* ink_drop_layer);
 
   // View:
   bool CanProcessEventsWithinSubtree() const override;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(InkDropContainerView);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_ANIMATION_INK_DROP_H_