Adds MD ink ripple animations to buttons within location bar

chrome/browser/ui/views/location_bar/bubble_icon_view.h

 // Copyright 2013 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 CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_BUBBLE_ICON_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_BUBBLE_ICON_VIEW_H_
 
 #include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
 #include "ui/gfx/image/image_skia.h"
 #include "ui/gfx/vector_icons_public.h"
+#include "ui/views/animation/ink_drop_host.h"
 #include "ui/views/controls/image_view.h"
+#include "ui/views/widget/widget_observer_view.h"
 
 class CommandUpdater;
 
 namespace gfx {
 enum class VectorIconId;
 }
 
 namespace views {
 class BubbleDelegateView;
+class InkDropDelegate;
 }
 
 // Represents an icon on the omnibox that shows a bubble when clicked.
-class BubbleIconView : public views::ImageView {
+class BubbleIconView : public views::WidgetObserverView,
+                       public views::InkDropHost {
  protected:
   enum ExecuteSource {
     EXECUTE_SOURCE_MOUSE,
     EXECUTE_SOURCE_KEYBOARD,
     EXECUTE_SOURCE_GESTURE,
   };
 
   BubbleIconView(CommandUpdater* command_updater, int command_id);
   ~BubbleIconView() override;
 
   // Returns true if a related bubble is showing.
   bool IsBubbleShowing() const;
 
+  // Sets the image that should be displayed in |image_|.
+  void SetImage(const gfx::ImageSkia* image_skia);
+
+  // Returns the image currently displayed which can be empty if not set.

[Peter Kasting, 2016/01/28 00:22:08]

Nit: Comma after "displayed"

[varkha, 2016/01/28 00:32:50]

Done.

+  // The returned image is owned by |image_|.
+  const gfx::ImageSkia& GetImage() const;
+
+  // Sets the tooltip text.
+  void SetTooltipText(const base::string16& tooltip);
+
   // Invoked prior to executing the command.
   virtual void OnExecuting(ExecuteSource execute_source) = 0;
 
-  // views::ImageView:
+  // views::View:
   void GetAccessibleState(ui::AXViewState* state) override;
   bool GetTooltipText(const gfx::Point& p, base::string16* tooltip) const
       override;
+  gfx::Size GetPreferredSize() const override;
+  void Layout() override;
   bool OnMousePressed(const ui::MouseEvent& event) override;
   void OnMouseReleased(const ui::MouseEvent& event) override;
   bool OnKeyPressed(const ui::KeyEvent& event) override;
   void ViewHierarchyChanged(
       const ViewHierarchyChangedDetails& details) override;
   void OnNativeThemeChanged(const ui::NativeTheme* theme) override;
 
   // ui::EventHandler:
   void OnGestureEvent(ui::GestureEvent* event) override;
 
+  // views::InkDropHost:
+  void AddInkDropLayer(ui::Layer* ink_drop_layer) override;
+  void RemoveInkDropLayer(ui::Layer* ink_drop_layer) override;
+
+  // views::WidgetObserver:
+  void OnWidgetVisibilityChanged(views::Widget* widget,
+                                 bool visible) override;
+
  protected:
   // Calls OnExecuting and runs |command_id_| with a valid |command_updater_|.
   virtual void ExecuteCommand(ExecuteSource source);
 
   // Returns the bubble instance for the icon.
   virtual views::BubbleDelegateView* GetBubble() const = 0;
 
   // Gets the given vector icon in the correct color and size based on |active|
   // and whether Chrome's in material design mode.
-  virtual gfx::VectorIconId GetVectorIcon() const = 0;
+  virtual gfx::VectorIconId GetVectorIcon() const;
 
   // Sets the image using a PNG from the resource bundle. Returns true if an
   // image was set, or false if the icon should use a vector asset. This only
   // exists for non-MD mode. TODO(estade): remove it.
   virtual bool SetRasterIcon();
 
   // views::View:
   void OnBoundsChanged(const gfx::Rect& previous_bounds) override;
 
+  // views::InkDropHost:
+  gfx::Point CalculateInkDropCenter() const override;
+  bool ShouldShowInkDropHover() const override;
+
   // Updates the icon after some state has changed.
   void UpdateIcon();
 
   // Sets the active state of the icon. An active icon will be displayed in a
   // "call to action" color.
   void SetActiveInternal(bool active);
 
   bool active() const { return active_; }
 
  private:
+  // The image shown in the button.
+  views::ImageView* image_;
+
   // The CommandUpdater for the Browser object that owns the location bar.
   CommandUpdater* command_updater_;
 
   // The command ID executed when the user clicks this icon.
   const int command_id_;
 
   // The active state. The precise definition of "active" is unique to each
   // subclass, but generally indicates that the associated feature is acting on
   // the web page.
   bool active_;
 
   // This is used to check if the bookmark bubble was showing during the mouse
   // pressed event. If this is true then the mouse released event is ignored to
   // prevent the bubble from reshowing.
   bool suppress_mouse_released_action_;
 
+  // Animation delegate for the ink drop ripple effect.
+  scoped_ptr<views::InkDropDelegate> ink_drop_delegate_;
+
   DISALLOW_COPY_AND_ASSIGN(BubbleIconView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_BUBBLE_ICON_VIEW_H_