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

Unified Diff: chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.h

Issue 393943006: MacViews: Remove GetMenu() from MenuRunner. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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/renderer_context_menu/render_view_context_menu_views.h
diff --git a/chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.h b/chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.h
index 3dcbaf03009f7343266336403b19b5ce20d2d035..778eb9f26e2d9019c2e07c97efad38f3247ae55b 100644
--- a/chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.h
+++ b/chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.h
@@ -16,6 +16,8 @@ class Point;
}
namespace views {
+class MenuItemView;
+class MenuModelAdapter;
class MenuRunner;
class Widget;
}
@@ -61,6 +63,10 @@ class RenderViewContextMenuViews : public RenderViewContextMenu {
ui::SimpleMenuModel bidi_submenu_model_;
scoped_ptr<views::MenuRunner> menu_runner_;
+ scoped_ptr<views::MenuModelAdapter> menu_adapter_;
sky 2014/07/15 23:53:00 I think this should be before menu_runner_.
Andre 2014/07/16 00:25:16 Done.
+
+ // Weak. Owned by menu_runner_;
+ views::MenuItemView* menu_view_;
sky 2014/07/15 23:53:00 Initialize to NULL.
Andre 2014/07/16 00:25:16 Done.
DISALLOW_COPY_AND_ASSIGN(RenderViewContextMenuViews);
};

Powered by Google App Engine
This is Rietveld 408576698