Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(774)

Unified Diff: chrome/browser/ui/views/toolbar/toolbar_action_view.h

Issue 1550443002: Pushed InkDropHost inheritence up to CustomButton. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge with master Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/toolbar/toolbar_action_view.h
diff --git a/chrome/browser/ui/views/toolbar/toolbar_action_view.h b/chrome/browser/ui/views/toolbar/toolbar_action_view.h
index a0dde7d9ecbf448b5616ae3f33bfd2ec831f44cf..699740102c291e9502f0be1f2e7f7a620c2cbc1d 100644
--- a/chrome/browser/ui/views/toolbar/toolbar_action_view.h
+++ b/chrome/browser/ui/views/toolbar/toolbar_action_view.h
@@ -8,7 +8,6 @@
#include "base/callback.h"
#include "base/macros.h"
#include "chrome/browser/ui/views/toolbar/toolbar_action_view_delegate_views.h"
-#include "ui/views/animation/ink_drop_host.h"
#include "ui/views/context_menu_controller.h"
#include "ui/views/controls/button/menu_button.h"
#include "ui/views/controls/button/menu_button_listener.h"
@@ -38,8 +37,7 @@ class MenuRunner;
class ToolbarActionView : public views::MenuButton,
public ToolbarActionViewDelegateViews,
public views::MenuButtonListener,
- public views::ContextMenuController,
- public views::InkDropHost {
+ public views::ContextMenuController {
public:
// Need DragController here because ToolbarActionView could be
// dragged/dropped.
@@ -78,6 +76,8 @@ class ToolbarActionView : public views::MenuButton,
scoped_ptr<views::LabelButtonBorder> CreateDefaultBorder() const override;
void OnMouseEntered(const ui::MouseEvent& event) override;
bool ShouldEnterPushedState(const ui::Event& event) override;
+ void AddInkDropLayer(ui::Layer* ink_drop_layer) override;
+ void RemoveInkDropLayer(ui::Layer* ink_drop_layer) override;
// ToolbarActionViewDelegateViews:
content::WebContents* GetCurrentWebContents() const override;
@@ -87,10 +87,6 @@ class ToolbarActionView : public views::MenuButton,
void OnMenuButtonClicked(views::View* sender,
const gfx::Point& point) override;
- // views::InkDropHost:
- void AddInkDropLayer(ui::Layer* ink_drop_layer) override;
- void RemoveInkDropLayer(ui::Layer* ink_drop_layer) override;
-
ToolbarActionViewController* view_controller() {
return view_controller_;
}
@@ -129,9 +125,6 @@ class ToolbarActionView : public views::MenuButton,
const gfx::Point& point,
ui::MenuSourceType source_type) override;
- // views::InkDropHost:
- gfx::Point CalculateInkDropCenter() const override;
-
// Shows the context menu (if one exists) for the toolbar action.
void DoShowContextMenu(ui::MenuSourceType source_type);
« no previous file with comments | « chrome/browser/ui/views/toolbar/app_menu_button.cc ('k') | chrome/browser/ui/views/toolbar/toolbar_action_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698