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

Side by Side Diff: chrome/browser/autocomplete/autocomplete_classifier_factory.cc

Issue 1165913002: [Cleanup] Used scoped pointers in KeyedServiceFactory's SetTestingFactory functions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Finish renaming profile -> context Created 5 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
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/autocomplete/autocomplete_classifier_factory.h" 5 #include "chrome/browser/autocomplete/autocomplete_classifier_factory.h"
6 6
7 #include "chrome/browser/autocomplete/autocomplete_classifier.h" 7 #include "chrome/browser/autocomplete/autocomplete_classifier.h"
8 #include "chrome/browser/autocomplete/autocomplete_controller.h" 8 #include "chrome/browser/autocomplete/autocomplete_controller.h"
9 #include "chrome/browser/autocomplete/chrome_autocomplete_scheme_classifier.h" 9 #include "chrome/browser/autocomplete/chrome_autocomplete_scheme_classifier.h"
10 #include "chrome/browser/autocomplete/in_memory_url_index_factory.h" 10 #include "chrome/browser/autocomplete/in_memory_url_index_factory.h"
(...skipping 14 matching lines...) Expand all
25 return static_cast<AutocompleteClassifier*>( 25 return static_cast<AutocompleteClassifier*>(
26 GetInstance()->GetServiceForBrowserContext(profile, true)); 26 GetInstance()->GetServiceForBrowserContext(profile, true));
27 } 27 }
28 28
29 // static 29 // static
30 AutocompleteClassifierFactory* AutocompleteClassifierFactory::GetInstance() { 30 AutocompleteClassifierFactory* AutocompleteClassifierFactory::GetInstance() {
31 return Singleton<AutocompleteClassifierFactory>::get(); 31 return Singleton<AutocompleteClassifierFactory>::get();
32 } 32 }
33 33
34 // static 34 // static
35 KeyedService* AutocompleteClassifierFactory::BuildInstanceFor( 35 scoped_ptr<KeyedService> AutocompleteClassifierFactory::BuildInstanceFor(
36 content::BrowserContext* context) { 36 content::BrowserContext* context) {
37 Profile* profile = static_cast<Profile*>(context); 37 Profile* profile = static_cast<Profile*>(context);
38 return new AutocompleteClassifier( 38 return make_scoped_ptr(new AutocompleteClassifier(
39 make_scoped_ptr(new AutocompleteController( 39 make_scoped_ptr(new AutocompleteController(
40 profile, TemplateURLServiceFactory::GetForProfile(profile), NULL, 40 profile, TemplateURLServiceFactory::GetForProfile(profile), NULL,
41 AutocompleteClassifier::kDefaultOmniboxProviders)), 41 AutocompleteClassifier::kDefaultOmniboxProviders)),
42 scoped_ptr<AutocompleteSchemeClassifier>( 42 scoped_ptr<AutocompleteSchemeClassifier>(
43 new ChromeAutocompleteSchemeClassifier(profile))); 43 new ChromeAutocompleteSchemeClassifier(profile))));
44 } 44 }
45 45
46 AutocompleteClassifierFactory::AutocompleteClassifierFactory() 46 AutocompleteClassifierFactory::AutocompleteClassifierFactory()
47 : BrowserContextKeyedServiceFactory( 47 : BrowserContextKeyedServiceFactory(
48 "AutocompleteClassifier", 48 "AutocompleteClassifier",
49 BrowserContextDependencyManager::GetInstance()) { 49 BrowserContextDependencyManager::GetInstance()) {
50 #if defined(ENABLE_EXTENSIONS) 50 #if defined(ENABLE_EXTENSIONS)
51 DependsOn( 51 DependsOn(
52 extensions::ExtensionsBrowserClient::Get()->GetExtensionSystemFactory()); 52 extensions::ExtensionsBrowserClient::Get()->GetExtensionSystemFactory());
53 #endif 53 #endif
(...skipping 11 matching lines...) Expand all
65 content::BrowserContext* context) const { 65 content::BrowserContext* context) const {
66 return chrome::GetBrowserContextRedirectedInIncognito(context); 66 return chrome::GetBrowserContextRedirectedInIncognito(context);
67 } 67 }
68 68
69 bool AutocompleteClassifierFactory::ServiceIsNULLWhileTesting() const { 69 bool AutocompleteClassifierFactory::ServiceIsNULLWhileTesting() const {
70 return true; 70 return true;
71 } 71 }
72 72
73 KeyedService* AutocompleteClassifierFactory::BuildServiceInstanceFor( 73 KeyedService* AutocompleteClassifierFactory::BuildServiceInstanceFor(
74 content::BrowserContext* profile) const { 74 content::BrowserContext* profile) const {
75 return BuildInstanceFor(static_cast<Profile*>(profile)); 75 return BuildInstanceFor(static_cast<Profile*>(profile)).release();
76 } 76 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698