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

Side by Side Diff: chrome/browser/download/download_service_factory.cc

Issue 10535113: Add dependencies on HistoryService/BookmarkModel. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 6 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 #include "chrome/browser/download/download_service_factory.h" 5 #include "chrome/browser/download/download_service_factory.h"
6 6
7 #include "chrome/browser/download/download_service.h" 7 #include "chrome/browser/download/download_service.h"
8 #include "chrome/browser/history/history_service_factory.h"
8 #include "chrome/browser/profiles/profile_dependency_manager.h" 9 #include "chrome/browser/profiles/profile_dependency_manager.h"
9 10
10 // static 11 // static
11 DownloadService* DownloadServiceFactory::GetForProfile( 12 DownloadService* DownloadServiceFactory::GetForProfile(
12 Profile* profile) { 13 Profile* profile) {
13 return static_cast<DownloadService*>( 14 return static_cast<DownloadService*>(
14 GetInstance()->GetServiceForProfile(profile, true)); 15 GetInstance()->GetServiceForProfile(profile, true));
15 } 16 }
16 17
17 // static 18 // static
18 DownloadServiceFactory* DownloadServiceFactory::GetInstance() { 19 DownloadServiceFactory* DownloadServiceFactory::GetInstance() {
19 return Singleton<DownloadServiceFactory>::get(); 20 return Singleton<DownloadServiceFactory>::get();
20 } 21 }
21 22
22 DownloadServiceFactory::DownloadServiceFactory() 23 DownloadServiceFactory::DownloadServiceFactory()
23 : ProfileKeyedServiceFactory("DownloadService", 24 : ProfileKeyedServiceFactory("DownloadService",
24 ProfileDependencyManager::GetInstance()) { 25 ProfileDependencyManager::GetInstance()) {
25 // TODO(rdsmith): For Shutdown() order we need to: 26 DependsOn(HistoryServiceFactory::GetInstance());
26 // DependsOn(HistoryServiceDataFactory::GetInstance());
27 } 27 }
28 28
29 DownloadServiceFactory::~DownloadServiceFactory() { 29 DownloadServiceFactory::~DownloadServiceFactory() {
30 } 30 }
31 31
32 ProfileKeyedService* DownloadServiceFactory::BuildServiceInstanceFor( 32 ProfileKeyedService* DownloadServiceFactory::BuildServiceInstanceFor(
33 Profile* profile) const { 33 Profile* profile) const {
34 DownloadService* service = new DownloadService(profile); 34 DownloadService* service = new DownloadService(profile);
35 35
36 // No need for initialization; initialization can be done on first 36 // No need for initialization; initialization can be done on first
37 // use of service. 37 // use of service.
38 38
39 return service; 39 return service;
40 } 40 }
41 41
42 bool DownloadServiceFactory::ServiceHasOwnInstanceInIncognito() { 42 bool DownloadServiceFactory::ServiceHasOwnInstanceInIncognito() {
43 return true; 43 return true;
44 } 44 }
OLDNEW
« no previous file with comments | « chrome/browser/bookmarks/bookmark_model_factory.cc ('k') | chrome/browser/predictors/autocomplete_action_predictor_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698