Implemented prototype for new ink drop specs.

chrome/browser/ui/views/toolbar/toolbar_button.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_TOOLBAR_TOOLBAR_BUTTON_H_
 #define CHROME_BROWSER_UI_VIEWS_TOOLBAR_TOOLBAR_BUTTON_H_
 
 #include "base/memory/scoped_ptr.h"
+#include "ui/gfx/geometry/point.h"
 #include "ui/views/animation/ink_drop_host.h"
 #include "ui/views/context_menu_controller.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/controls/button/label_button.h"
 
 namespace ui {
 class MenuModel;
 }
 
 namespace views {
@@ skipping 42 matching lines @@
   void AddInkDropLayer(ui::Layer* ink_drop_layer) override;
   void RemoveInkDropLayer(ui::Layer* ink_drop_layer) override;
 
  protected:
   // Overridden from CustomButton. Returns true if the button should become
   // pressed when a user holds the mouse down over the button. For this
   // implementation, both left and right mouse buttons can trigger a change
   // to the PUSHED state.
   bool ShouldEnterPushedState(const ui::Event& event) override;
 
+  // views::LabelButton:

[Peter Kasting, 2015/09/09 22:50:46]

Nit: I would put this heading over both the overrides here and eliminate the
comment above entirely:

  // views::LabelButton:
  bool ShouldEnterPushedState(const ui::Event& event) override;
  void NotifyClick(const ui::Event& event) override;

Any commentary on ShouldEnterPushedState can be given in the implementation.

[bruthig, 2015/09/10 14:57:38]

Done.

+  void NotifyClick(const ui::Event& event) override;
+
   // Returns if menu should be shown. Override this to change default behavior.
   virtual bool ShouldShowMenu();
 
   // Function to show the dropdown menu.
   virtual void ShowDropDownMenu(ui::MenuSourceType source_type);
 
-  virtual void LayoutInkDrop();
+  // Returns the Point where the ink drop should be centered.
+  virtual gfx::Point CalculateInkDropCenter() const;
 
   views::InkDropAnimationController* ink_drop_animation_controller() {
     return ink_drop_animation_controller_.get();
   }
 
  private:
   // views::LabelButton:
   const char* GetClassName() const override;
 
   // The model that populates the attached menu.
@@ skipping 11 matching lines @@
   // Animation controller for the ink drop ripple effect.
   scoped_ptr<views::InkDropAnimationController> ink_drop_animation_controller_;
 
   // A factory for tasks that show the dropdown context menu for the button.
   base::WeakPtrFactory<ToolbarButton> show_menu_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ToolbarButton);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TOOLBAR_TOOLBAR_BUTTON_H_