WIP Component build of autofill

chrome/browser/webdata/web_data_service_factory.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/webdata/web_data_service_factory.h"
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile_dependency_manager.h"
@@ skipping 100 matching lines @@
 scoped_refptr<AutofillWebDataService>
 WebDataServiceWrapper::GetAutofillWebData() {
   return autofill_web_data_.get();
 }
 
 scoped_refptr<WebDataService> WebDataServiceWrapper::GetWebData() {
   return web_data_.get();
 }
 
 // static
+scoped_refptr<WebDataService>
+WebDataServiceFactory::GetWebDataServiceForProfile(
+    Profile* profile) {
+  // 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.
+  WebDataServiceWrapper* wds_wrapper =
+      WebDataServiceFactory::GetForProfile(
+          profile, Profile::EXPLICIT_ACCESS, true);
+  if (wds_wrapper)
+    return wds_wrapper->GetWebData();
+  return NULL;
+}
+
+// static
+scoped_refptr<WebDataService>
+WebDataServiceFactory::GetWebDataServiceForProfileIfExists(Profile* profile) {
+  // 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.
+  WebDataServiceWrapper* wds_wrapper =
+      WebDataServiceFactory::GetForProfile(
+          profile, Profile::EXPLICIT_ACCESS, false);
+  if (wds_wrapper)
+    return wds_wrapper->GetWebData();
+  return NULL;
+}
+
+// static
 scoped_refptr<AutofillWebDataService>
-AutofillWebDataService::FromBrowserContext(content::BrowserContext* context) {
+WebDataServiceFactory::GetAutofillWebDataServiceForProfile(Profile* profile) {
   // 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.
-  WebDataServiceWrapper* wrapper =
+  WebDataServiceWrapper* wds_wrapper =
       WebDataServiceFactory::GetForProfile(
-          static_cast<Profile*>(context), Profile::EXPLICIT_ACCESS);
-  if (wrapper)
-    return wrapper->GetAutofillWebData();
-  // |wrapper| can be NULL in Incognito mode.
-  return scoped_refptr<AutofillWebDataService>(NULL);
-}
-
-// static
-scoped_refptr<WebDataService> WebDataService::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.
-  WebDataServiceWrapper* wrapper =
-      WebDataServiceFactory::GetForProfile(
-          static_cast<Profile*>(context), Profile::EXPLICIT_ACCESS);
-  if (wrapper)
-    return wrapper->GetWebData();
-  // |wrapper| can be NULL in Incognito mode.
-  return scoped_refptr<WebDataService>(NULL);
+          profile, Profile::EXPLICIT_ACCESS, true);
+  if (wds_wrapper)
+    return wds_wrapper->GetAutofillWebData();
+  return NULL;
 }
 
 WebDataServiceFactory::WebDataServiceFactory()
     : ProfileKeyedServiceFactory("WebDataService",
                                  ProfileDependencyManager::GetInstance()) {
   // WebDataServiceFactory has no dependecies.
 }
 
 WebDataServiceFactory::~WebDataServiceFactory() {}
 
 // static
-WebDataServiceWrapper* WebDataServiceFactory::GetForProfile(
-    Profile* profile, Profile::ServiceAccessType access_type) {
-  // If |access_type| starts being used for anything other than this
-  // DCHECK, we need to start taking it as a parameter to
-  // AutofillWebDataService::FromBrowserContext (see above).
-  DCHECK(access_type != Profile::IMPLICIT_ACCESS || !profile->IsOffTheRecord());
-  return static_cast<WebDataServiceWrapper*>(
-          GetInstance()->GetServiceForProfile(profile, true));
-}
-
-// static
-WebDataServiceWrapper* WebDataServiceFactory::GetForProfileIfExists(
-    Profile* profile, Profile::ServiceAccessType access_type) {
-  // If |access_type| starts being used for anything other than this
-  // DCHECK, we need to start taking it as a parameter to
-  // AutofillWebDataService::FromBrowserContext (see above).
-  DCHECK(access_type != Profile::IMPLICIT_ACCESS || !profile->IsOffTheRecord());
-  return static_cast<WebDataServiceWrapper*>(
-          GetInstance()->GetServiceForProfile(profile, false));
-}
-
-// static
 WebDataServiceFactory* WebDataServiceFactory::GetInstance() {
   return Singleton<WebDataServiceFactory>::get();
 }
 
+// static
+WebDataServiceWrapper* WebDataServiceFactory::GetForProfile(
+    Profile* profile,
+    Profile::ServiceAccessType access_type,
+    bool create_if_not_exist) {
+  // If |access_type| starts being used for anything other than this DCHECK, we
+  // need to start taking it as a parameter to GetXXXForProfile functions (see
+  // above).
+  DCHECK(access_type != Profile::IMPLICIT_ACCESS || !profile->IsOffTheRecord());
+  return static_cast<WebDataServiceWrapper*>(
+      GetInstance()->GetServiceForProfile(profile, create_if_not_exist));
+}
+
 bool WebDataServiceFactory::ServiceRedirectedInIncognito() const {
   return true;
 }
 
 ProfileKeyedService*
 WebDataServiceFactory::BuildServiceInstanceFor(Profile* profile) const {
   return new WebDataServiceWrapper(profile);
 }
 
 bool WebDataServiceFactory::ServiceIsNULLWhileTesting() const {
   return true;
 }