Index: chrome/browser/ui/cocoa/wrench_menu/wrench_menu_controller.mm |
diff --git a/chrome/browser/ui/cocoa/wrench_menu/wrench_menu_controller.mm b/chrome/browser/ui/cocoa/wrench_menu/wrench_menu_controller.mm |
index 9b7458b4186971cbf9c9c3dc0805f1a524f6c272..467be2a12dd3af3120a8abc4b1b0fd34bd10ca72 100644 |
--- a/chrome/browser/ui/cocoa/wrench_menu/wrench_menu_controller.mm |
+++ b/chrome/browser/ui/cocoa/wrench_menu/wrench_menu_controller.mm |
@@ -25,10 +25,10 @@ |
#import "chrome/browser/ui/cocoa/toolbar/toolbar_controller.h" |
#import "chrome/browser/ui/cocoa/wrench_menu/menu_tracked_root_view.h" |
#import "chrome/browser/ui/cocoa/wrench_menu/recent_tabs_menu_model_delegate.h" |
+#include "chrome/browser/ui/toolbar/app_menu_model.h" |
#include "chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h" |
#include "chrome/browser/ui/toolbar/toolbar_actions_bar.h" |
#include "chrome/browser/ui/toolbar/toolbar_actions_bar_observer.h" |
-#include "chrome/browser/ui/toolbar/wrench_menu_model.h" |
#include "chrome/grit/generated_resources.h" |
#include "components/ui/zoom/zoom_event_manager.h" |
#include "content/public/browser/user_metrics.h" |
@@ -100,10 +100,10 @@ class ZoomLevelObserver { |
private: |
void OnZoomLevelChanged(const content::HostZoomMap::ZoomLevelChange& change) { |
- WrenchMenuModel* wrenchMenuModel = [controller_ wrenchMenuModel]; |
- wrenchMenuModel->UpdateZoomControls(); |
+ AppMenuModel* appMenuModel = [controller_ appMenuModel]; |
+ appMenuModel->UpdateZoomControls(); |
const base::string16 level = |
- wrenchMenuModel->GetLabelForCommandId(IDC_ZOOM_PERCENT_DISPLAY); |
+ appMenuModel->GetLabelForCommandId(IDC_ZOOM_PERCENT_DISPLAY); |
[[controller_ zoomDisplay] setTitle:SysUTF16ToNSString(level)]; |
} |
@@ -163,7 +163,7 @@ class ToolbarActionsBarObserverHelper : public ToolbarActionsBarObserver { |
// a weak Browser*, or pass this call onto reference counted objects. |
recentTabsMenuModelDelegate_.reset(); |
[self setModel:nullptr]; |
- wrenchMenuModel_.reset(); |
+ appMenuModel_.reset(); |
buttonViewController_.reset(); |
// The observers should most likely already be destroyed (since they're reset |
@@ -287,9 +287,8 @@ class ToolbarActionsBarObserverHelper : public ToolbarActionsBarObserver { |
NSMenu* bookmarkMenu = [self bookmarkSubMenu]; |
DCHECK(bookmarkMenu); |
- bookmarkMenuBridge_.reset( |
- new BookmarkMenuBridge([self wrenchMenuModel]->browser()->profile(), |
- bookmarkMenu)); |
+ bookmarkMenuBridge_.reset(new BookmarkMenuBridge( |
+ [self appMenuModel]->browser()->profile(), bookmarkMenu)); |
} |
- (void)updateBrowserActionsSubmenu { |
@@ -339,14 +338,14 @@ class ToolbarActionsBarObserverHelper : public ToolbarActionsBarObserver { |
ui_zoom::ZoomEventManager::GetForBrowserContext( |
browser_->profile()))); |
NSString* title = base::SysUTF16ToNSString( |
- [self wrenchMenuModel]->GetLabelForCommandId(IDC_ZOOM_PERCENT_DISPLAY)); |
+ [self appMenuModel]->GetLabelForCommandId(IDC_ZOOM_PERCENT_DISPLAY)); |
[[[buttonViewController_ zoomItem] viewWithTag:IDC_ZOOM_PERCENT_DISPLAY] |
setTitle:title]; |
content::RecordAction(UserMetricsAction("ShowAppMenu")); |
- NSImage* icon = [self wrenchMenuModel]->browser()->window()->IsFullscreen() ? |
- [NSImage imageNamed:NSImageNameExitFullScreenTemplate] : |
- [NSImage imageNamed:NSImageNameEnterFullScreenTemplate]; |
+ NSImage* icon = [self appMenuModel]->browser()->window()->IsFullscreen() |
+ ? [NSImage imageNamed:NSImageNameExitFullScreenTemplate] |
+ : [NSImage imageNamed:NSImageNameEnterFullScreenTemplate]; |
[[buttonViewController_ zoomFullScreen] setImage:icon]; |
} |
@@ -413,12 +412,12 @@ class ToolbarActionsBarObserverHelper : public ToolbarActionsBarObserver { |
// Used to perform the actual dispatch on the outermost runloop. |
- (void)performCommandDispatch:(NSNumber*)tag { |
- [self wrenchMenuModel]->ExecuteCommand([tag intValue], 0); |
+ [self appMenuModel]->ExecuteCommand([tag intValue], 0); |
} |
-- (WrenchMenuModel*)wrenchMenuModel { |
- // Don't use |wrenchMenuModel_| so that a test can override the generic one. |
- return static_cast<WrenchMenuModel*>(model_); |
+- (AppMenuModel*)appMenuModel { |
+ // Don't use |appMenuModel_| so that a test can override the generic one. |
+ return static_cast<AppMenuModel*>(model_); |
} |
- (void)updateRecentTabsSubmenu { |
@@ -436,9 +435,8 @@ class ToolbarActionsBarObserverHelper : public ToolbarActionsBarObserver { |
- (void)createModel { |
DCHECK(browser_); |
recentTabsMenuModelDelegate_.reset(); |
- wrenchMenuModel_.reset( |
- new WrenchMenuModel(acceleratorDelegate_.get(), browser_)); |
- [self setModel:wrenchMenuModel_.get()]; |
+ appMenuModel_.reset(new AppMenuModel(acceleratorDelegate_.get(), browser_)); |
+ [self setModel:appMenuModel_.get()]; |
buttonViewController_.reset( |
[[WrenchMenuButtonViewController alloc] initWithController:self]); |
@@ -525,7 +523,7 @@ class ToolbarActionsBarObserverHelper : public ToolbarActionsBarObserver { |
int index = 0; |
// Start searching at the wrench menu model level, |model| will be updated |
// only if the command we're looking for is found in one of the [sub]menus. |
- ui::MenuModel* model = [self wrenchMenuModel]; |
+ ui::MenuModel* model = [self appMenuModel]; |
if (ui::MenuModel::GetModelAndIndexForCommandId( |
RecentTabsSubMenuModel::kRecentlyClosedHeaderCommandId, &model, |
&index)) { |