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

Side by Side Diff: chrome/browser/ui/cocoa/extensions/extension_action_context_menu.mm

Issue 11246003: Remove Profile->GetExtensionProcessManager (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix extension_crash_recovery_browsertest Created 8 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import "chrome/browser/ui/cocoa/extensions/extension_action_context_menu.h" 5 #import "chrome/browser/ui/cocoa/extensions/extension_action_context_menu.h"
6 6
7 #include "base/sys_string_conversions.h" 7 #include "base/sys_string_conversions.h"
8 #include "chrome/browser/api/prefs/pref_change_registrar.h" 8 #include "chrome/browser/api/prefs/pref_change_registrar.h"
9 #include "chrome/browser/extensions/extension_action.h" 9 #include "chrome/browser/extensions/extension_action.h"
10 #include "chrome/browser/extensions/extension_action_manager.h" 10 #include "chrome/browser/extensions/extension_action_manager.h"
11 #include "chrome/browser/extensions/extension_service.h" 11 #include "chrome/browser/extensions/extension_service.h"
12 #include "chrome/browser/extensions/extension_system.h"
12 #include "chrome/browser/extensions/extension_tab_util.h" 13 #include "chrome/browser/extensions/extension_tab_util.h"
13 #include "chrome/browser/extensions/extension_uninstall_dialog.h" 14 #include "chrome/browser/extensions/extension_uninstall_dialog.h"
14 #include "chrome/browser/prefs/pref_service.h" 15 #include "chrome/browser/prefs/pref_service.h"
15 #include "chrome/browser/profiles/profile.h" 16 #include "chrome/browser/profiles/profile.h"
16 #include "chrome/browser/ui/browser.h" 17 #include "chrome/browser/ui/browser.h"
17 #include "chrome/browser/ui/browser_finder.h" 18 #include "chrome/browser/ui/browser_finder.h"
18 #include "chrome/browser/ui/browser_tabstrip.h" 19 #include "chrome/browser/ui/browser_tabstrip.h"
19 #include "chrome/browser/ui/chrome_pages.h" 20 #include "chrome/browser/ui/chrome_pages.h"
20 #include "chrome/browser/ui/cocoa/browser_window_cocoa.h" 21 #include "chrome/browser/ui/cocoa/browser_window_cocoa.h"
21 #include "chrome/browser/ui/cocoa/browser_window_controller.h" 22 #include "chrome/browser/ui/cocoa/browser_window_controller.h"
(...skipping 132 matching lines...) Expand 10 before | Expand all | Expand 10 after
154 GURL url(std::string(extension_urls::kGalleryBrowsePrefix) + 155 GURL url(std::string(extension_urls::kGalleryBrowsePrefix) +
155 std::string("/detail/") + extension_->id()); 156 std::string("/detail/") + extension_->id());
156 OpenURLParams params( 157 OpenURLParams params(
157 url, Referrer(), NEW_FOREGROUND_TAB, content::PAGE_TRANSITION_LINK, 158 url, Referrer(), NEW_FOREGROUND_TAB, content::PAGE_TRANSITION_LINK,
158 false); 159 false);
159 browser_->OpenURL(params); 160 browser_->OpenURL(params);
160 break; 161 break;
161 } 162 }
162 case kExtensionContextOptions: { 163 case kExtensionContextOptions: {
163 DCHECK(!extension_->options_url().is_empty()); 164 DCHECK(!extension_->options_url().is_empty());
164 browser_->profile()->GetExtensionProcessManager()->OpenOptionsPage( 165 extensions::ExtensionSystem::Get(browser_->profile())->process_manager()->
165 extension_, browser_); 166 OpenOptionsPage(extension_, browser_);
166 break; 167 break;
167 } 168 }
168 case kExtensionContextDisable: { 169 case kExtensionContextDisable: {
169 ExtensionService* extensionService = 170 ExtensionService* extensionService =
170 browser_->profile()->GetExtensionService(); 171 browser_->profile()->GetExtensionService();
171 if (!extensionService) 172 if (!extensionService)
172 return; // Incognito mode. 173 return; // Incognito mode.
173 extensionService->DisableExtension(extension_->id(), 174 extensionService->DisableExtension(extension_->id(),
174 Extension::DISABLE_USER_ACTION); 175 Extension::DISABLE_USER_ACTION);
175 break; 176 break;
(...skipping 21 matching lines...) Expand all
197 198
198 - (BOOL)validateMenuItem:(NSMenuItem*)menuItem { 199 - (BOOL)validateMenuItem:(NSMenuItem*)menuItem {
199 if ([menuItem tag] == kExtensionContextOptions) { 200 if ([menuItem tag] == kExtensionContextOptions) {
200 // Disable 'Options' if there are no options to set. 201 // Disable 'Options' if there are no options to set.
201 return extension_->options_url().spec().length() > 0; 202 return extension_->options_url().spec().length() > 0;
202 } 203 }
203 return YES; 204 return YES;
204 } 205 }
205 206
206 @end 207 @end
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698