diff options
| author | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-11 23:51:20 +0000 |
|---|---|---|
| committer | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-11 23:51:20 +0000 |
| commit | 14831ef6c6fb54bd7aafa1006c5a2650ee6e9097 (patch) | |
| tree | 7fb7b566271922e09a25997488b2d9137d50a051 /views/examples/menu_example.h | |
| parent | 755fd4806a674e254387e3053857b5475453c8b8 (diff) | |
| download | chromium_src-14831ef6c6fb54bd7aafa1006c5a2650ee6e9097.zip chromium_src-14831ef6c6fb54bd7aafa1006c5a2650ee6e9097.tar.gz chromium_src-14831ef6c6fb54bd7aafa1006c5a2650ee6e9097.tar.bz2 | |
Re-organize RenderViewContextMenu to the use cross-platform MenuModel class.
Benefits:
- less code
- more straightforward code
- more shared code
unresolved issues:
- RenderViewContextMenu{Gtk,Mac,Views} should not be a subclass of RenderViewContextMenu
- On Linux, IMEs are added in the wrong place (not near the other editable stuff)
BUG=31365
TEST=manual + trybots
Review URL: http://codereview.chromium.org/2017001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46987 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views/examples/menu_example.h')
| -rw-r--r-- | views/examples/menu_example.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/views/examples/menu_example.h b/views/examples/menu_example.h index 07d27d1..d43acde 100644 --- a/views/examples/menu_example.h +++ b/views/examples/menu_example.h @@ -57,7 +57,7 @@ class ExampleMenuContents : public menus::SimpleMenuModel, submenu_.reset(new menus::SimpleMenuModel(this)); submenu_->AddItem(kCommandDoSomething, WideToUTF16(L"Do Something 2")); - AddSubMenu(ASCIIToUTF16("Submenu"), submenu_.get()); + AddSubMenu(-1, ASCIIToUTF16("Submenu"), submenu_.get()); menu_.reset(new views::Menu2(this)); } |