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

Unified Diff: views/controls/menu/menu_model_adapter.cc

Issue 7720012: Moves ownership of MenuItemView to MenuRunner as well as responbility (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix unit test Created 9 years, 4 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: views/controls/menu/menu_model_adapter.cc
diff --git a/views/controls/menu/menu_model_adapter.cc b/views/controls/menu/menu_model_adapter.cc
index e2c895d4349148823ef769d5ae9bccdccb90a06c..850b6b5e9806a587f6574ef95a52fedda343dc5d 100644
--- a/views/controls/menu/menu_model_adapter.cc
+++ b/views/controls/menu/menu_model_adapter.cc
@@ -44,6 +44,12 @@ void MenuModelAdapter::BuildMenu(MenuItemView* menu) {
menu->ChildrenChanged();
}
+MenuItemView* MenuModelAdapter::CreateMenu() {
+ MenuItemView* item = new MenuItemView(this);
+ BuildMenu(item);
+ return item;
+}
+
// MenuModelAdapter, MenuDelegate implementation:
void MenuModelAdapter::ExecuteCommand(int id) {

Powered by Google App Engine
This is Rietveld 408576698