diff options
| author | viettrungluu@chromium.org <viettrungluu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-05 23:38:34 +0000 |
|---|---|---|
| committer | viettrungluu@chromium.org <viettrungluu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-05 23:38:34 +0000 |
| commit | 3bb0b82f159c8784cd71a371d106eaefd97114e9 (patch) | |
| tree | 0adab0ba4b78f9caa42be944849a4ee0825dfbc7 /chrome/browser/cocoa/menu_button_unittest.mm | |
| parent | 9dc32ca678b8295b5a68cd1ddf90467c8a4502cf (diff) | |
| download | chromium_src-3bb0b82f159c8784cd71a371d106eaefd97114e9.zip chromium_src-3bb0b82f159c8784cd71a371d106eaefd97114e9.tar.gz chromium_src-3bb0b82f159c8784cd71a371d106eaefd97114e9.tar.bz2 | |
Mac: Eliminate unintended "menu" method overrides.
This had the bad effect of showing bad/strange menus upon right-click on various buttons or, worse, crashing.
BUG=22148,23606
TEST=Right click on back/forwards buttons (after some navigation so that they're active), page/wrench buttons, and bookmark bar off-the-side button and make sure no menu is shown (and it that it doesn't crash). Also do so when the menu is active/highlighted with menu already showing.
Review URL: http://codereview.chromium.org/251091
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28064 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/cocoa/menu_button_unittest.mm')
| -rw-r--r-- | chrome/browser/cocoa/menu_button_unittest.mm | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/chrome/browser/cocoa/menu_button_unittest.mm b/chrome/browser/cocoa/menu_button_unittest.mm index 1861974..321a3bb4 100644 --- a/chrome/browser/cocoa/menu_button_unittest.mm +++ b/chrome/browser/cocoa/menu_button_unittest.mm @@ -51,14 +51,14 @@ TEST_F(MenuButtonTest, MenuAssign) { [menu insertItemWithTitle:@"bar" action:nil keyEquivalent:@"" atIndex:2]; [menu insertItemWithTitle:@"baz" action:nil keyEquivalent:@"" atIndex:3]; - [button_ setMenu:menu]; - EXPECT_TRUE([button_ menu]); + [button_ setAttachedMenu:menu]; + EXPECT_TRUE([button_ attachedMenu]); // TODO(viettrungluu): Display the menu. (The tough part is closing the menu, // not opening it!) // Since |button_| doesn't retain menu, we should probably unset it here. - [button_ setMenu:nil]; + [button_ setAttachedMenu:nil]; } } // namespace |