Remove GenericRequest templates from WebDataService

chrome/browser/webdata/web_data_service.h

 // 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.
 
 // Chromium settings and storage represent user-selected preferences and
 // information and MUST not be extracted, overwritten or modified except
 // through Chromium defined APIs.
 
 #ifndef CHROME_BROWSER_WEBDATA_WEB_DATA_SERVICE_H__
 #define CHROME_BROWSER_WEBDATA_WEB_DATA_SERVICE_H__
 
 #include <map>
 #include <string>
 #include <vector>
 
+#include "base/callback.h"

[erikwright (departed), 2013/01/14 15:51:36]

I don't see why this is required here, but if it is, the subsequent include of
callback_forward should be removed.

Try moving this to the .cc file.

 #include "base/callback_forward.h"
 #include "base/file_path.h"
 #include "base/location.h"
 #include "base/memory/ref_counted.h"
 #include "base/sequenced_task_runner_helpers.h"
 #include "base/synchronization/lock.h"
 #include "chrome/browser/api/webdata/autofill_web_data_service.h"
 #include "chrome/browser/api/webdata/web_data_results.h"
 #include "chrome/browser/api/webdata/web_data_service_base.h"
 #include "chrome/browser/api/webdata/web_data_service_consumer.h"
@@ skipping 37 matching lines @@
 
 
 ////////////////////////////////////////////////////////////////////////////////
 //
 // WebDataService results
 //
 ////////////////////////////////////////////////////////////////////////////////
 
 typedef std::vector<AutofillChange> AutofillChangeList;
 
+typedef base::Callback<scoped_ptr<WDTypedResult>(void)> ResultTask;
+
 // Result from GetWebAppImages.
 struct WDAppImagesResult {
   WDAppImagesResult();
   ~WDAppImagesResult();
 
   // True if SetWebAppHasAllImages(true) was invoked.
   bool has_all_images;
 
   // The images, may be empty.
   std::vector<SkBitmap> images;
@@ skipping 248 matching lines @@
   //////////////////////////////////////////////////////////////////////////////
   //
   // The following methods are only invoked in the web data service thread.
   //
   //////////////////////////////////////////////////////////////////////////////
  private:
   friend struct content::BrowserThread::DeleteOnThread<
       content::BrowserThread::UI>;
   friend class base::DeleteHelper<WebDataService>;
 
-  typedef GenericRequest2<std::vector<const TemplateURLData*>,
-                          KeywordTable::Keywords> SetKeywordsRequest;
-
   // Invoked on the main thread if initializing the db fails.
   void DBInitFailed(sql::InitStatus init_status);
 
   // Initialize the database, if it hasn't already been initialized.
   void InitializeDatabaseIfNecessary();
 
   // Initialize any syncable services.
   void InitializeSyncableServices();
 
   // The notification method.
   void NotifyDatabaseLoadedOnUIThread();
 
   // Commit any pending transaction and deletes the database.
   void ShutdownDatabase();
 
   // Deletes the syncable services.
   void ShutdownSyncableServices();
 
   // Commit the current transaction and creates a new one.
   void Commit();
 
   // Schedule a task on our worker thread.
   void ScheduleTask(const tracked_objects::Location& from_here,
                     const base::Closure& task);
 
+  void ScheduleDBTask(const tracked_objects::Location& from_here,
+                      const base::Closure& task);
+
+  WebDataService::Handle ScheduleDBTaskWithResult(
+      const tracked_objects::Location& from_here,
+      const ResultTask& task,
+      WebDataServiceConsumer* consumer);
+
+  void DBTaskWrapper(const base::Closure& task,
+                     WebDataRequest* request);
+
+  void DBResultTaskWrapper(const ResultTask& task,
+                           WebDataRequest* request);
+
   // Schedule a commit if one is not already pending.
   void ScheduleCommit();
 
   //////////////////////////////////////////////////////////////////////////////
   //
   // Keywords.
   //
   //////////////////////////////////////////////////////////////////////////////
-  void AddKeywordImpl(GenericRequest<TemplateURLData>* request);
-  void RemoveKeywordImpl(GenericRequest<TemplateURLID>* request);
-  void UpdateKeywordImpl(GenericRequest<TemplateURLData>* request);
-  void GetKeywordsImpl(WebDataRequest* request);
-  void SetDefaultSearchProviderImpl(GenericRequest<TemplateURLID>* r);
-  void SetBuiltinKeywordVersionImpl(GenericRequest<int>* r);
+  void AddKeywordImpl(const TemplateURLData& data);
+  void RemoveKeywordImpl(TemplateURLID id);
+  void UpdateKeywordImpl(const TemplateURLData& data);
+  scoped_ptr<WDTypedResult> GetKeywordsImpl();
+  void SetDefaultSearchProviderImpl(TemplateURLID r);
+  void SetBuiltinKeywordVersionImpl(int version);
 
   //////////////////////////////////////////////////////////////////////////////
   //
   // Web Apps.
   //
   //////////////////////////////////////////////////////////////////////////////
-  void SetWebAppImageImpl(GenericRequest2<GURL, SkBitmap>* request);
-  void SetWebAppHasAllImagesImpl(GenericRequest2<GURL, bool>* request);
-  void RemoveWebAppImpl(GenericRequest<GURL>* request);
-  void GetWebAppImagesImpl(GenericRequest<GURL>* request);
+
+  void SetWebAppImageImpl(const GURL& app_url, const SkBitmap& image);
+  void SetWebAppHasAllImagesImpl(const GURL& app_url, bool has_all_images);
+  void RemoveWebAppImpl(const GURL& app_url);
+  scoped_ptr<WDTypedResult> GetWebAppImagesImpl(const GURL& app_url);
 
   //////////////////////////////////////////////////////////////////////////////
   //
   // Web Intents.
   //
   //////////////////////////////////////////////////////////////////////////////
   void AddWebIntentServiceImpl(
-      GenericRequest<webkit_glue::WebIntentServiceData>* request);
+      const webkit_glue::WebIntentServiceData& service);
   void RemoveWebIntentServiceImpl(
-      GenericRequest<webkit_glue::WebIntentServiceData>* request);
-  void GetWebIntentServicesImpl(GenericRequest<string16>* request);
-  void GetWebIntentServicesForURLImpl(GenericRequest<string16>* request);
-  void GetAllWebIntentServicesImpl(GenericRequest<std::string>* request);
-  void AddDefaultWebIntentServiceImpl(
-      GenericRequest<DefaultWebIntentService>* request);
+      const webkit_glue::WebIntentServiceData& service);
+  scoped_ptr<WDTypedResult> GetWebIntentServicesImpl(const string16& action);
+  scoped_ptr<WDTypedResult> GetWebIntentServicesForURLImpl(
+      const string16& service_url);
+  scoped_ptr<WDTypedResult> GetAllWebIntentServicesImpl();
+  void AddDefaultWebIntentServiceImpl(const DefaultWebIntentService& service);
   void RemoveDefaultWebIntentServiceImpl(
-      GenericRequest<DefaultWebIntentService>* request);
-  void RemoveWebIntentServiceDefaultsImpl(GenericRequest<GURL>* request);
-  void GetDefaultWebIntentServicesForActionImpl(
-      GenericRequest<string16>* request);
-  void GetAllDefaultWebIntentServicesImpl(GenericRequest<std::string>* request);
+      const DefaultWebIntentService& service);
+  void RemoveWebIntentServiceDefaultsImpl(const GURL& service_url);
+  scoped_ptr<WDTypedResult> GetDefaultWebIntentServicesForActionImpl(
+      const string16& action);
+  scoped_ptr<WDTypedResult> GetAllDefaultWebIntentServicesImpl();
 
   //////////////////////////////////////////////////////////////////////////////
   //
   // Token Service.
   //
   //////////////////////////////////////////////////////////////////////////////
 
-  void RemoveAllTokensImpl(GenericRequest<std::string>* request);
-  void SetTokenForServiceImpl(
-    GenericRequest2<std::string, std::string>* request);
-  void GetAllTokensImpl(GenericRequest<std::string>* request);
+  void RemoveAllTokensImpl();
+  void SetTokenForServiceImpl(const std::string& service,
+                              const std::string& token);
+  scoped_ptr<WDTypedResult> GetAllTokensImpl();
 
 #if defined(OS_WIN)
   //////////////////////////////////////////////////////////////////////////////
   //
   // Password manager.
   //
   //////////////////////////////////////////////////////////////////////////////
-  void AddIE7LoginImpl(GenericRequest<IE7PasswordInfo>* request);
-  void RemoveIE7LoginImpl(GenericRequest<IE7PasswordInfo>* request);
-  void GetIE7LoginImpl(GenericRequest<IE7PasswordInfo>* request);
+  void AddIE7LoginImpl(const IE7PasswordInfo& info);
+  void RemoveIE7LoginImpl(const IE7PasswordInfo& info);
+  scoped_ptr<WDTypedResult> GetIE7LoginImpl(const IE7PasswordInfo& info);
 #endif  // defined(OS_WIN)
 
   //////////////////////////////////////////////////////////////////////////////
   //
   // Autofill.
   //
   //////////////////////////////////////////////////////////////////////////////
-  void AddFormElementsImpl(
-      GenericRequest<std::vector<FormFieldData> >* request);
-  void GetFormValuesForElementNameImpl(WebDataRequest* request,
+  void AddFormElementsImpl(const std::vector<FormFieldData>& fields);
+  scoped_ptr<WDTypedResult> GetFormValuesForElementNameImpl(
       const string16& name, const string16& prefix, int limit);
   void RemoveFormElementsAddedBetweenImpl(
-      GenericRequest2<base::Time, base::Time>* request);
-  void RemoveExpiredFormElementsImpl(WebDataRequest* request);
-  void RemoveFormValueForElementNameImpl(
-      GenericRequest2<string16, string16>* request);
-  void AddAutofillProfileImpl(GenericRequest<AutofillProfile>* request);
-  void UpdateAutofillProfileImpl(GenericRequest<AutofillProfile>* request);
-  void RemoveAutofillProfileImpl(GenericRequest<std::string>* request);
-  void GetAutofillProfilesImpl(WebDataRequest* request);
-  void EmptyMigrationTrashImpl(GenericRequest<bool>* request);
-  void AddCreditCardImpl(GenericRequest<CreditCard>* request);
-  void UpdateCreditCardImpl(GenericRequest<CreditCard>* request);
-  void RemoveCreditCardImpl(GenericRequest<std::string>* request);
-  void GetCreditCardsImpl(WebDataRequest* request);
+      const base::Time& delete_begin, const base::Time& delete_end);
+  void RemoveExpiredFormElementsImpl();
+  void RemoveFormValueForElementNameImpl(const string16& name,
+                                         const string16& value);
+  void AddAutofillProfileImpl(const AutofillProfile& profile);
+  void UpdateAutofillProfileImpl(const AutofillProfile& profile);
+  void RemoveAutofillProfileImpl(const std::string& guid);
+  scoped_ptr<WDTypedResult> GetAutofillProfilesImpl();
+  void EmptyMigrationTrashImpl(bool notify_sync);
+  void AddCreditCardImpl(const CreditCard& credit_card);
+  void UpdateCreditCardImpl(const CreditCard& credit_card);
+  void RemoveCreditCardImpl(const std::string& guid);
+  scoped_ptr<WDTypedResult> GetCreditCardsImpl();
   void RemoveAutofillProfilesAndCreditCardsModifiedBetweenImpl(
-      GenericRequest2<base::Time, base::Time>* request);
+      const base::Time& delete_begin, const base::Time& delete_end);
 
   // Callbacks to ensure that sensitive info is destroyed if request is
   // cancelled.
   void DestroyAutofillProfileResult(const WDTypedResult* result);
   void DestroyAutofillCreditCardResult(const WDTypedResult* result);
 
   // True once initialization has started.
   bool is_running_;
 
   // The path with which to initialize the database.
@@ skipping 21 matching lines @@
   // Whether we should commit the database.
   bool should_commit_;
 
   // MessageLoop the WebDataService is created on.
   MessageLoop* main_loop_;
 
   DISALLOW_COPY_AND_ASSIGN(WebDataService);
 };
 
 #endif  // CHROME_BROWSER_WEBDATA_WEB_DATA_SERVICE_H__