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

Unified Diff: chrome/browser/webdata/web_data_service_factory.cc

Issue 12851008: Create a common base class for all the webdatas (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix upstream branch Created 7 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/webdata/web_data_service_factory.cc
diff --git a/chrome/browser/webdata/web_data_service_factory.cc b/chrome/browser/webdata/web_data_service_factory.cc
index 81dbc43f5c78684eb0c30d171a18b0690e9d6c1a..0dcb1e100d2dd197f4835e9e37dfa1469088d19a 100644
--- a/chrome/browser/webdata/web_data_service_factory.cc
+++ b/chrome/browser/webdata/web_data_service_factory.cc
@@ -31,8 +31,8 @@ scoped_refptr<WebDataService> WebDataServiceWrapper::GetWebData() {
}
// static
-scoped_ptr<AutofillWebDataService> AutofillWebDataService::FromBrowserContext(
- content::BrowserContext* context) {
+scoped_refptr<AutofillWebDataService>
+AutofillWebDataService::FromBrowserContext(content::BrowserContext* context) {
// For this service, the implicit/explicit distinction doesn't
// really matter; it's just used for a DCHECK. So we currently
// cheat and always say EXPLICIT_ACCESS.
@@ -40,10 +40,10 @@ scoped_ptr<AutofillWebDataService> AutofillWebDataService::FromBrowserContext(
static_cast<Profile*>(context), Profile::EXPLICIT_ACCESS);
if (service.get()) {
- return scoped_ptr<AutofillWebDataService>(
+ return scoped_refptr<AutofillWebDataService>(
new AutofillWebDataServiceImpl(service));
} else {
- return scoped_ptr<AutofillWebDataService>(NULL);
+ return scoped_refptr<AutofillWebDataService>(NULL);
}
}

Powered by Google App Engine
This is Rietveld 408576698