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

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: 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 142919f7237ae7a463095b214bb7d312aa8de8ac..48621e26276636fd25ab6cb6f05b580aa906e64e 100644
--- a/chrome/browser/webdata/web_data_service_factory.cc
+++ b/chrome/browser/webdata/web_data_service_factory.cc
@@ -29,7 +29,7 @@ scoped_refptr<WebDataService> WebDataServiceWrapper::GetWebData() {
}
// static
-scoped_ptr<AutofillWebDataService> AutofillWebDataService::FromBrowserContext(
+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
@@ -38,10 +38,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