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

Side by Side Diff: chrome/browser/extensions/menu_manager_factory.cc

Issue 459493002: Revert of Extend contextMenus API to support browser/page actions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 #include "chrome/browser/extensions/menu_manager_factory.h" 5 #include "chrome/browser/extensions/menu_manager_factory.h"
6 6
7 #include "chrome/browser/extensions/menu_manager.h" 7 #include "chrome/browser/extensions/menu_manager.h"
8 #include "chrome/browser/profiles/profile.h" 8 #include "chrome/browser/profiles/profile.h"
9 #include "components/keyed_service/content/browser_context_dependency_manager.h" 9 #include "components/keyed_service/content/browser_context_dependency_manager.h"
10 #include "extensions/browser/extension_system.h" 10 #include "extensions/browser/extension_system.h"
11 #include "extensions/browser/extension_system_provider.h" 11 #include "extensions/browser/extension_system_provider.h"
12 #include "extensions/browser/extensions_browser_client.h" 12 #include "extensions/browser/extensions_browser_client.h"
13 13
14 namespace extensions { 14 namespace extensions {
15 15
16 // static 16 // static
17 MenuManager* MenuManagerFactory::GetForBrowserContext( 17 MenuManager* MenuManagerFactory::GetForBrowserContext(
18 content::BrowserContext* context) { 18 content::BrowserContext* context) {
19 return static_cast<MenuManager*>( 19 return static_cast<MenuManager*>(
20 GetInstance()->GetServiceForBrowserContext(context, true)); 20 GetInstance()->GetServiceForBrowserContext(context, true));
21 } 21 }
22 22
23 // static 23 // static
24 MenuManagerFactory* MenuManagerFactory::GetInstance() { 24 MenuManagerFactory* MenuManagerFactory::GetInstance() {
25 return Singleton<MenuManagerFactory>::get(); 25 return Singleton<MenuManagerFactory>::get();
26 } 26 }
27 27
28 // static
29 KeyedService* MenuManagerFactory::BuildServiceInstanceForTesting(
30 content::BrowserContext* context) {
31 return GetInstance()->BuildServiceInstanceFor(context);
32 }
33
34 MenuManagerFactory::MenuManagerFactory() 28 MenuManagerFactory::MenuManagerFactory()
35 : BrowserContextKeyedServiceFactory( 29 : BrowserContextKeyedServiceFactory(
36 "MenuManager", 30 "MenuManager",
37 BrowserContextDependencyManager::GetInstance()) { 31 BrowserContextDependencyManager::GetInstance()) {
38 DependsOn(ExtensionsBrowserClient::Get()->GetExtensionSystemFactory()); 32 DependsOn(ExtensionsBrowserClient::Get()->GetExtensionSystemFactory());
39 } 33 }
40 34
41 MenuManagerFactory::~MenuManagerFactory() {} 35 MenuManagerFactory::~MenuManagerFactory() {}
42 36
43 KeyedService* MenuManagerFactory::BuildServiceInstanceFor( 37 KeyedService* MenuManagerFactory::BuildServiceInstanceFor(
44 content::BrowserContext* context) const { 38 content::BrowserContext* context) const {
45 Profile* profile = Profile::FromBrowserContext(context); 39 Profile* profile = Profile::FromBrowserContext(context);
46 return new MenuManager(profile, ExtensionSystem::Get(profile)->state_store()); 40 return new MenuManager(
41 profile,
42 ExtensionSystem::Get(profile)->state_store());
47 } 43 }
48 44
49 content::BrowserContext* MenuManagerFactory::GetBrowserContextToUse( 45 content::BrowserContext* MenuManagerFactory::GetBrowserContextToUse(
50 content::BrowserContext* context) const { 46 content::BrowserContext* context) const {
51 return ExtensionsBrowserClient::Get()->GetOriginalContext(context); 47 return ExtensionsBrowserClient::Get()->GetOriginalContext(context);
52 } 48 }
53 49
54 bool MenuManagerFactory::ServiceIsCreatedWithBrowserContext() const { 50 bool MenuManagerFactory::ServiceIsCreatedWithBrowserContext() const {
55 return true; 51 return true;
56 } 52 }
57 53
58 bool MenuManagerFactory::ServiceIsNULLWhileTesting() const { 54 bool MenuManagerFactory::ServiceIsNULLWhileTesting() const {
59 return true; 55 return true;
60 } 56 }
61 57
62 } // namespace extensions 58 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/browser/extensions/menu_manager_factory.h ('k') | chrome/browser/renderer_context_menu/render_view_context_menu.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698