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

Unified Diff: chrome/browser/cocoa/back_forward_menu_controller.mm

Issue 251091: Mac: Eliminate unintended "menu" method overrides. (Closed)
Patch Set: Created 11 years, 2 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/cocoa/back_forward_menu_controller.mm
diff --git a/chrome/browser/cocoa/back_forward_menu_controller.mm b/chrome/browser/cocoa/back_forward_menu_controller.mm
index a826df5eab71c93aeef0733fc20a8fd9a3c1a186..a362f492ffe2d074650514367c9b8241c667aee5 100644
--- a/chrome/browser/cocoa/back_forward_menu_controller.mm
+++ b/chrome/browser/cocoa/back_forward_menu_controller.mm
@@ -30,37 +30,37 @@ using gfx::SkBitmapToNSImage;
button_ = button;
model_.reset(new BackForwardMenuModel(browser, type_));
DCHECK(model_.get());
- menu_.reset([[NSMenu alloc] initWithTitle:@""]);
- DCHECK(menu_.get());
- [menu_ setDelegate:self];
+ backForwardMenu_.reset([[NSMenu alloc] initWithTitle:@""]);
+ DCHECK(backForwardMenu_.get());
+ [backForwardMenu_ setDelegate:self];
- [button_ setMenu:menu_];
- [button_ setMenuEnabled:YES];
+ [button_ setAttachedMenu:backForwardMenu_];
+ [button_ setAttachedMenuEnabled:YES];
}
return self;
}
// Methods as delegate:
-// Called by menu_ just before tracking begins.
+// Called by backForwardMenu_ just before tracking begins.
//TODO(viettrungluu@gmail.com): do anything for chapter stops (see model)?
- (void)menuNeedsUpdate:(NSMenu*)menu {
- DCHECK(menu == menu_);
+ DCHECK(menu == backForwardMenu_);
// Remove old menu items (backwards order is as good as any).
- for (NSInteger i = [menu_ numberOfItems]; i > 0; i--)
- [menu_ removeItemAtIndex:(i-1)];
+ for (NSInteger i = [menu numberOfItems]; i > 0; i--)
+ [menu removeItemAtIndex:(i-1)];
// 0-th item must be blank. (This is because we use a pulldown list, for which
// Cocoa uses the 0-th item as "title" in the button.)
- [menu_ insertItemWithTitle:@""
- action:nil
- keyEquivalent:@""
- atIndex:0];
+ [menu insertItemWithTitle:@""
+ action:nil
+ keyEquivalent:@""
+ atIndex:0];
for (int menuID = 1; menuID <= model_->GetTotalItemCount(); menuID++) {
if (model_->IsSeparator(menuID)) {
- [menu_ insertItem:[NSMenuItem separatorItem]
- atIndex:menuID];
+ [menu insertItem:[NSMenuItem separatorItem]
+ atIndex:menuID];
} else {
// Create a menu item with the right label.
NSMenuItem* menuItem = [[NSMenuItem alloc]
@@ -83,8 +83,8 @@ using gfx::SkBitmapToNSImage;
[menuItem setAction:@selector(executeMenuItem:)];
// Put it in the menu!
- [menu_ insertItem:menuItem
- atIndex:menuID];
+ [menu insertItem:menuItem
+ atIndex:menuID];
}
}
}
« no previous file with comments | « chrome/browser/cocoa/back_forward_menu_controller.h ('k') | chrome/browser/cocoa/bookmark_bar_controller.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698