Index: chrome/browser/extensions/context_menu_matcher.cc |
diff --git a/chrome/browser/extensions/context_menu_matcher.cc b/chrome/browser/extensions/context_menu_matcher.cc |
index 18a0058d5b60204ca6c05a449a9f2b41ac308884..81cb9712a037365e008541abf347eb4338ace606 100644 |
--- a/chrome/browser/extensions/context_menu_matcher.cc |
+++ b/chrome/browser/extensions/context_menu_matcher.cc |
@@ -6,30 +6,18 @@ |
#include "base/strings/utf_string_conversions.h" |
#include "chrome/app/chrome_command_ids.h" |
-#include "chrome/browser/extensions/extension_service.h" |
#include "chrome/browser/extensions/extension_util.h" |
+#include "chrome/common/extensions/api/context_menus.h" |
#include "content/public/browser/browser_context.h" |
#include "content/public/common/context_menu_params.h" |
-#include "extensions/browser/extension_system.h" |
+#include "extensions/browser/extension_registry.h" |
#include "ui/gfx/favicon_size.h" |
#include "ui/gfx/image/image.h" |
-#if defined(ENABLE_EXTENSIONS) |
-#include "chrome/common/extensions/api/context_menus.h" |
-#endif |
- |
namespace extensions { |
namespace { |
-int GetActionMenuTopLevelLimit() { |
-#if defined(ENABLE_EXTENSIONS) |
- return api::context_menus::ACTION_MENU_TOP_LEVEL_LIMIT; |
-#else |
- return 0; |
-#endif |
-} |
- |
// The range of command IDs reserved for extension's custom menus. |
// TODO(oshima): These values will be injected by embedders. |
int extensions_context_custom_first = IDC_EXTENSIONS_CONTEXT_CUSTOM_FIRST; |
@@ -77,7 +65,7 @@ void ContextMenuMatcher::AppendExtensionItems( |
MenuItem::List items; |
bool can_cross_incognito; |
if (!GetRelevantExtensionTopLevelItems( |
- extension_key, &extension, &can_cross_incognito, items)) |
+ extension_key, &extension, &can_cross_incognito, &items)) |
return; |
if (items.empty()) |
@@ -151,7 +139,7 @@ base::string16 ContextMenuMatcher::GetTopLevelContextMenuTitle( |
MenuItem::List items; |
bool can_cross_incognito; |
GetRelevantExtensionTopLevelItems( |
- extension_key, &extension, &can_cross_incognito, items); |
+ extension_key, &extension, &can_cross_incognito, &items); |
base::string16 title; |
@@ -196,11 +184,10 @@ bool ContextMenuMatcher::GetRelevantExtensionTopLevelItems( |
const MenuItem::ExtensionKey& extension_key, |
const Extension** extension, |
bool* can_cross_incognito, |
- MenuItem::List& items) { |
- ExtensionService* service = |
- extensions::ExtensionSystem::Get(browser_context_)->extension_service(); |
- *extension = service->GetExtensionById(extension_key.extension_id, false); |
- |
+ MenuItem::List* items) { |
+ *extension = ExtensionRegistry::Get( |
+ browser_context_)->enabled_extensions().GetByID( |
+ extension_key.extension_id); |
if (!*extension) |
return false; |
@@ -211,8 +198,7 @@ bool ContextMenuMatcher::GetRelevantExtensionTopLevelItems( |
return false; |
*can_cross_incognito = util::CanCrossIncognito(*extension, browser_context_); |
- items = GetRelevantExtensionItems(*all_items, |
- *can_cross_incognito); |
+ *items = GetRelevantExtensionItems(*all_items, *can_cross_incognito); |
return true; |
} |
@@ -264,8 +250,9 @@ void ContextMenuMatcher::RecursivelyAppendExtensionItems( |
// Action context menus have a limit for top level extension items to |
// prevent control items from being pushed off the screen, since extension |
// items will not be placed in a submenu. |
+ const int top_level_limit = api::context_menus::ACTION_MENU_TOP_LEVEL_LIMIT; |
if (menu_id >= extensions_context_custom_last || |
- (is_action_menu_top_level && num_items >= GetActionMenuTopLevelLimit())) |
+ (is_action_menu_top_level && num_items >= top_level_limit)) |
return; |
extension_item_map_[menu_id] = item->id(); |