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

Side by Side Diff: extensions/browser/process_manager_factory.cc

Issue 2863213004: ProcessManagerFactory: fix dependencies in ApiResourceManager, LazyBgndTaskFactory (Closed)
Patch Set: Add missing dependencies from ApiResourceManager to ExtensionRegistryFactory, ProcessManagerFactory Created 3 years, 7 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
« no previous file with comments | « extensions/browser/browser_context_keyed_api_factory.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "extensions/browser/process_manager_factory.h"
benwells 2017/05/10 16:00:23 Nit: blank link after this include.
Kevin M 2017/05/10 17:19:36 Done.
5 #include "components/keyed_service/content/browser_context_dependency_manager.h" 6 #include "components/keyed_service/content/browser_context_dependency_manager.h"
6 #include "extensions/browser/extension_registry_factory.h" 7 #include "extensions/browser/extension_registry_factory.h"
8 #include "extensions/browser/lazy_background_task_queue_factory.h"
7 #include "extensions/browser/process_manager.h" 9 #include "extensions/browser/process_manager.h"
8 #include "extensions/browser/process_manager_factory.h"
9 10
10 using content::BrowserContext; 11 using content::BrowserContext;
11 12
12 namespace extensions { 13 namespace extensions {
13 14
14 // static 15 // static
15 ProcessManager* ProcessManagerFactory::GetForBrowserContext( 16 ProcessManager* ProcessManagerFactory::GetForBrowserContext(
16 BrowserContext* context) { 17 BrowserContext* context) {
17 return static_cast<ProcessManager*>( 18 return static_cast<ProcessManager*>(
18 GetInstance()->GetServiceForBrowserContext(context, true)); 19 GetInstance()->GetServiceForBrowserContext(context, true));
19 } 20 }
20 21
21 // static 22 // static
22 ProcessManager* ProcessManagerFactory::GetForBrowserContextIfExists( 23 ProcessManager* ProcessManagerFactory::GetForBrowserContextIfExists(
23 BrowserContext* context) { 24 BrowserContext* context) {
24 return static_cast<ProcessManager*>( 25 return static_cast<ProcessManager*>(
25 GetInstance()->GetServiceForBrowserContext(context, false)); 26 GetInstance()->GetServiceForBrowserContext(context, false));
26 } 27 }
27 28
28 // static 29 // static
29 ProcessManagerFactory* ProcessManagerFactory::GetInstance() { 30 ProcessManagerFactory* ProcessManagerFactory::GetInstance() {
30 return base::Singleton<ProcessManagerFactory>::get(); 31 return base::Singleton<ProcessManagerFactory>::get();
31 } 32 }
32 33
33 ProcessManagerFactory::ProcessManagerFactory() 34 ProcessManagerFactory::ProcessManagerFactory()
34 : BrowserContextKeyedServiceFactory( 35 : BrowserContextKeyedServiceFactory(
35 "ProcessManager", 36 "ProcessManager",
36 BrowserContextDependencyManager::GetInstance()) { 37 BrowserContextDependencyManager::GetInstance()) {
37 DependsOn(extensions::ExtensionRegistryFactory::GetInstance()); 38 DependsOn(extensions::ExtensionRegistryFactory::GetInstance());
39 DependsOn(extensions::LazyBackgroundTaskQueueFactory::GetInstance());
38 } 40 }
39 41
40 ProcessManagerFactory::~ProcessManagerFactory() { 42 ProcessManagerFactory::~ProcessManagerFactory() {
41 } 43 }
42 44
43 KeyedService* ProcessManagerFactory::BuildServiceInstanceFor( 45 KeyedService* ProcessManagerFactory::BuildServiceInstanceFor(
44 BrowserContext* context) const { 46 BrowserContext* context) const {
45 return ProcessManager::Create(context); 47 return ProcessManager::Create(context);
46 } 48 }
47 49
48 BrowserContext* ProcessManagerFactory::GetBrowserContextToUse( 50 BrowserContext* ProcessManagerFactory::GetBrowserContextToUse(
49 BrowserContext* context) const { 51 BrowserContext* context) const {
50 // ProcessManager::Create handles guest and incognito profiles, returning an 52 // ProcessManager::Create handles guest and incognito profiles, returning an
51 // IncognitoProcessManager in incognito mode. 53 // IncognitoProcessManager in incognito mode.
52 return context; 54 return context;
53 } 55 }
54 56
55 } // namespace extensions 57 } // namespace extensions
OLDNEW
« no previous file with comments | « extensions/browser/browser_context_keyed_api_factory.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698