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

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

Issue 1766023002: [Extensions UI Views] Adjust the wrench menu actions for items added/removed (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2661
Patch Set: Created 4 years, 10 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/extension_toolbar_menu_view.h
diff --git a/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h b/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h
index e192668fe76fe8287ebb48b8738ed3edfc69d898..bba9cba88bae785d878e8f1b1b257d2c8105c472 100644
--- a/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h
+++ b/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h
@@ -16,6 +16,10 @@ class Browser;
class BrowserActionsContainer;
class ToolbarActionsBar;
+namespace views {
+class MenuItemView;
+}
+
// ExtensionToolbarMenuView is the view containing the extension actions that
// overflowed from the BrowserActionsContainer, and is contained in and owned by
// the app menu.
@@ -24,7 +28,9 @@ class ToolbarActionsBar;
class ExtensionToolbarMenuView : public views::ScrollView,
public ToolbarActionsBarObserver {
public:
- ExtensionToolbarMenuView(Browser* browser, AppMenu* app_menu);
+ ExtensionToolbarMenuView(Browser* browser,
+ AppMenu* app_menu,
+ views::MenuItemView* menu_item);
~ExtensionToolbarMenuView() override;
// Returns whether the app menu should show this view. This is true when
@@ -66,6 +72,9 @@ class ExtensionToolbarMenuView : public views::ScrollView,
// The app menu, which may need to be closed after a drag-and-drop.
AppMenu* app_menu_;
+ // The MenuItemView this view is contained within.
+ views::MenuItemView* menu_item_;
+
// The overflow BrowserActionsContainer which is nested in this view.
BrowserActionsContainer* container_;
« no previous file with comments | « chrome/browser/ui/views/toolbar/app_menu.cc ('k') | chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698