Separate WebDataRequest functionality from WebDataService

chrome/browser/webdata/web_data_service.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.h"
 
 #include "base/bind.h"
 #include "base/message_loop.h"
 #include "base/stl_util.h"
 #include "base/threading/thread.h"
@@ skipping 66 matching lines @@
 
 WDKeywordsResult::~WDKeywordsResult() {}
 
 WebDataService::WebDataService()
     : is_running_(false),
       db_(NULL),
       autocomplete_syncable_service_(NULL),
       autofill_profile_syncable_service_(NULL),
       failed_init_(false),
       should_commit_(false),
-      next_request_handle_(1),
       main_loop_(MessageLoop::current()) {
   // WebDataService requires DB thread if instantiated.
   // Set WebDataServiceFactory::GetInstance()->SetTestingFactory(&profile, NULL)
   // if you do not want to instantiate WebDataService in your test.
   DCHECK(BrowserThread::IsWellKnownThread(BrowserThread::DB));
+
+  wdr_mgr_.reset(new WebDataRequestManager());
 }
 
 // static
 void WebDataService::NotifyOfMultipleAutofillChanges(
     WebDataService* web_data_service) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
 
   if (!web_data_service)
     return;
 
@@ skipping 17 matching lines @@
 
 bool WebDataService::IsRunning() const {
   return is_running_;
 }
 
 void WebDataService::UnloadDatabase() {
   ScheduleTask(FROM_HERE, Bind(&WebDataService::ShutdownDatabase, this));
 }
 
 void WebDataService::CancelRequest(Handle h) {
-  base::AutoLock l(pending_lock_);
-  RequestMap::iterator i = pending_requests_.find(h);
-  if (i == pending_requests_.end()) {
-    NOTREACHED() << "Canceling a nonexistent web data service request";
-    return;
-  }
-  i->second->Cancel();
+  wdr_mgr_->CancelRequest(h);
 }
 
 content::NotificationSource WebDataService::GetNotificationSource() {
   return content::Source<WebDataService>(this);
 }
 
 bool WebDataService::IsDatabaseLoaded() {
   return db_ != NULL;
 }
 
 WebDatabase* WebDataService::GetDatabase() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
   return db_;
 }
 
 //////////////////////////////////////////////////////////////////////////////
 //
 // Keywords.
 //
 //////////////////////////////////////////////////////////////////////////////
 
 void WebDataService::AddKeyword(const TemplateURLData& data) {
   GenericRequest<TemplateURLData>* request =
-      new GenericRequest<TemplateURLData>(this, GetNextRequestHandle(), NULL,
-                                          data);
-  RegisterRequest(request);
+      new GenericRequest<TemplateURLData>(
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, data);
+  wdr_mgr_->RegisterRequest(request);

[dhollowa, 2013/01/04 01:34:50]

The common pattern here is:
  (1) new GenericRequest<Foo>(wdr_mgr->GetNextRequestHandle()),...
  (2) wdr_mgr_->RegisterRequest(request)

It would be preferable now to simply pass the wdr_mrg into the constructor of
the request, and let the request get its own handle and register itself.  This
will remove a tonne of boilerplate code in this file.

[Cait (Slow), 2013/01/04 22:59:29]

Done.

   ScheduleTask(FROM_HERE, Bind(&WebDataService::AddKeywordImpl, this, request));
 }
 
 void WebDataService::RemoveKeyword(TemplateURLID id) {
   GenericRequest<TemplateURLID>* request =
-      new GenericRequest<TemplateURLID>(this, GetNextRequestHandle(), NULL, id);
-  RegisterRequest(request);
+      new GenericRequest<TemplateURLID>(
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, id);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::RemoveKeywordImpl, this, request));
 }
 
 void WebDataService::UpdateKeyword(const TemplateURLData& data) {
   GenericRequest<TemplateURLData>* request =
-      new GenericRequest<TemplateURLData>(this, GetNextRequestHandle(), NULL,
-                                          data);
-  RegisterRequest(request);
+      new GenericRequest<TemplateURLData>(
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, data);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::UpdateKeywordImpl, this, request));
 }
 
 WebDataService::Handle WebDataService::GetKeywords(
                                        WebDataServiceConsumer* consumer) {
   WebDataRequest* request =
-      new WebDataRequest(this, GetNextRequestHandle(), consumer);
-  RegisterRequest(request);
+      new WebDataRequest(this, wdr_mgr_->GetNextRequestHandle(), consumer);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::GetKeywordsImpl, this, request));
   return request->GetHandle();
 }
 
 void WebDataService::SetDefaultSearchProvider(const TemplateURL* url) {
   GenericRequest<TemplateURLID>* request = new GenericRequest<TemplateURLID>(
-      this, GetNextRequestHandle(), NULL, url ? url->id() : 0);
-  RegisterRequest(request);
+      this, wdr_mgr_->GetNextRequestHandle(), NULL, url ? url->id() : 0);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE, Bind(&WebDataService::SetDefaultSearchProviderImpl,
                                this, request));
 }
 
 void WebDataService::SetBuiltinKeywordVersion(int version) {
-  GenericRequest<int>* request =
-      new GenericRequest<int>(this, GetNextRequestHandle(), NULL, version);
-  RegisterRequest(request);
+  GenericRequest<int>* request = new GenericRequest<int>(
+      this, wdr_mgr_->GetNextRequestHandle(), NULL, version);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE, Bind(&WebDataService::SetBuiltinKeywordVersionImpl,
                                this, request));
 }
 
 //////////////////////////////////////////////////////////////////////////////
 //
 // Web Apps
 //
 //////////////////////////////////////////////////////////////////////////////
 
 void WebDataService::SetWebAppImage(const GURL& app_url,
                                     const SkBitmap& image) {
   GenericRequest2<GURL, SkBitmap>* request =
-      new GenericRequest2<GURL, SkBitmap>(this, GetNextRequestHandle(),
-                                         NULL, app_url, image);
-  RegisterRequest(request);
+      new GenericRequest2<GURL, SkBitmap>(
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, app_url, image);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::SetWebAppImageImpl, this, request));
 }
 
 void WebDataService::SetWebAppHasAllImages(const GURL& app_url,
                                            bool has_all_images) {
   GenericRequest2<GURL, bool>* request =
-      new GenericRequest2<GURL, bool>(this, GetNextRequestHandle(),
-                                     NULL, app_url, has_all_images);
-  RegisterRequest(request);
+      new GenericRequest2<GURL, bool>(
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, app_url,
+          has_all_images);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::SetWebAppHasAllImagesImpl, this, request));
 }
 
 void WebDataService::RemoveWebApp(const GURL& app_url) {
   GenericRequest<GURL>* request =
-      new GenericRequest<GURL>(this, GetNextRequestHandle(), NULL, app_url);
-  RegisterRequest(request);
+      new GenericRequest<GURL>(
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, app_url);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::RemoveWebAppImpl, this, request));
 }
 
 WebDataService::Handle WebDataService::GetWebAppImages(
     const GURL& app_url,
     WebDataServiceConsumer* consumer) {
   GenericRequest<GURL>* request =
-      new GenericRequest<GURL>(this, GetNextRequestHandle(), consumer, app_url);
-  RegisterRequest(request);
+      new GenericRequest<GURL>(
+          this, wdr_mgr_->GetNextRequestHandle(), consumer, app_url);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::GetWebAppImagesImpl, this, request));
   return request->GetHandle();
 }
 
 //////////////////////////////////////////////////////////////////////////////
 //
 // Web Intents.
 //
 //////////////////////////////////////////////////////////////////////////////
 
 void WebDataService::AddWebIntentService(const WebIntentServiceData& service) {
   GenericRequest<WebIntentServiceData>* request =
       new GenericRequest<WebIntentServiceData>(
-          this, GetNextRequestHandle(), NULL, service);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, service);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::AddWebIntentServiceImpl, this, request));
 }
 
 void WebDataService::RemoveWebIntentService(
     const WebIntentServiceData& service) {
   GenericRequest<WebIntentServiceData>* request =
       new GenericRequest<WebIntentServiceData>(
-          this, GetNextRequestHandle(), NULL, service);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, service);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE, Bind(&WebDataService::RemoveWebIntentServiceImpl,
                                this, request));
 }
 
 WebDataService::Handle WebDataService::GetWebIntentServicesForAction(
     const string16& action,
     WebDataServiceConsumer* consumer) {
   DCHECK(consumer);
-  GenericRequest<string16>* request = new GenericRequest<string16>(
-      this, GetNextRequestHandle(), consumer, action);
-  RegisterRequest(request);
+  GenericRequest<string16>* request =
+      new GenericRequest<string16>(
+          this, wdr_mgr_->GetNextRequestHandle(), consumer, action);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::GetWebIntentServicesImpl, this, request));
   return request->GetHandle();
 }
 
 WebDataService::Handle WebDataService::GetWebIntentServicesForURL(
     const string16& service_url,
     WebDataServiceConsumer* consumer) {
   DCHECK(consumer);
-  GenericRequest<string16>* request = new GenericRequest<string16>(
-          this, GetNextRequestHandle(), consumer, service_url);
-  RegisterRequest(request);
+  GenericRequest<string16>* request =
+      new GenericRequest<string16>(
+          this, wdr_mgr_->GetNextRequestHandle(), consumer, service_url);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE, Bind(&WebDataService::GetWebIntentServicesForURLImpl,
                                this, request));
   return request->GetHandle();
 }
 
 
 WebDataService::Handle WebDataService::GetAllWebIntentServices(
     WebDataServiceConsumer* consumer) {
   DCHECK(consumer);
-  GenericRequest<std::string>* request = new GenericRequest<std::string>(
-      this, GetNextRequestHandle(), consumer, std::string());
-  RegisterRequest(request);
+  GenericRequest<std::string>* request =
+      new GenericRequest<std::string>(
+          this, wdr_mgr_->GetNextRequestHandle(), consumer, std::string());
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE, Bind(&WebDataService::GetAllWebIntentServicesImpl,
                                this, request));
   return request->GetHandle();
 }
 
 void WebDataService::AddDefaultWebIntentService(
     const DefaultWebIntentService& service) {
   GenericRequest<DefaultWebIntentService>* request =
       new GenericRequest<DefaultWebIntentService>(
-          this, GetNextRequestHandle(), NULL, service);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, service);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::AddDefaultWebIntentServiceImpl, this,
                     request));
 }
 
 void WebDataService::RemoveDefaultWebIntentService(
     const DefaultWebIntentService& service) {
   GenericRequest<DefaultWebIntentService>* request =
       new GenericRequest<DefaultWebIntentService>(
-          this, GetNextRequestHandle(), NULL, service);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, service);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::RemoveDefaultWebIntentServiceImpl, this,
                     request));
 }
 
 void WebDataService::RemoveWebIntentServiceDefaults(
     const GURL& service_url) {
   GenericRequest<GURL>* request =
       new GenericRequest<GURL>(
-          this, GetNextRequestHandle(), NULL, service_url);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, service_url);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(
       FROM_HERE,
       Bind(&WebDataService::RemoveWebIntentServiceDefaultsImpl, this, request));
 }
 
 WebDataService::Handle WebDataService::GetDefaultWebIntentServicesForAction(
     const string16& action,
     WebDataServiceConsumer* consumer) {
   DCHECK(consumer);
   GenericRequest<string16>* request = new GenericRequest<string16>(
-          this, GetNextRequestHandle(), consumer, action);
-  RegisterRequest(request);
+      this, wdr_mgr_->GetNextRequestHandle(), consumer, action);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::GetDefaultWebIntentServicesForActionImpl,
                     this, request));
   return request->GetHandle();
 }
 
 WebDataService::Handle WebDataService::GetAllDefaultWebIntentServices(
     WebDataServiceConsumer* consumer) {
   DCHECK(consumer);
   GenericRequest<std::string>* request = new GenericRequest<std::string>(
-      this, GetNextRequestHandle(), consumer, std::string());
-  RegisterRequest(request);
+      this, wdr_mgr_->GetNextRequestHandle(), consumer, std::string());
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::GetAllDefaultWebIntentServicesImpl,
                     this, request));
   return request->GetHandle();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 //
 // Token Service
 //
 ////////////////////////////////////////////////////////////////////////////////
 
 void WebDataService::SetTokenForService(const std::string& service,
                                         const std::string& token) {
   GenericRequest2<std::string, std::string>* request =
       new GenericRequest2<std::string, std::string>(
-          this, GetNextRequestHandle(), NULL, service, token);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, service, token);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::SetTokenForServiceImpl, this, request));
 }
 
 void WebDataService::RemoveAllTokens() {
   GenericRequest<std::string>* request =
       new GenericRequest<std::string>(
-          this, GetNextRequestHandle(), NULL, std::string());
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, std::string());
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::RemoveAllTokensImpl, this, request));
 }
 
 // Null on failure. Success is WDResult<std::string>
 WebDataService::Handle WebDataService::GetAllTokens(
     WebDataServiceConsumer* consumer) {
 
   GenericRequest<std::string>* request =
       new GenericRequest<std::string>(
-          this, GetNextRequestHandle(), consumer, std::string());
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), consumer, std::string());
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::GetAllTokensImpl, this, request));
   return request->GetHandle();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 //
 // Autofill.
 //
 ////////////////////////////////////////////////////////////////////////////////
 
 void WebDataService::AddFormFields(
     const std::vector<FormFieldData>& fields) {
   GenericRequest<std::vector<FormFieldData> >* request =
       new GenericRequest<std::vector<FormFieldData> >(
-          this, GetNextRequestHandle(), NULL, fields);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, fields);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::AddFormElementsImpl, this, request));
 }
 
 WebDataService::Handle WebDataService::GetFormValuesForElementName(
     const string16& name, const string16& prefix, int limit,
     WebDataServiceConsumer* consumer) {
   WebDataRequest* request =
-      new WebDataRequest(this, GetNextRequestHandle(), consumer);
-  RegisterRequest(request);
+      new WebDataRequest(this, wdr_mgr_->GetNextRequestHandle(), consumer);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::GetFormValuesForElementNameImpl,
                     this, request, name, prefix, limit));
   return request->GetHandle();
 }
 
 void WebDataService::RemoveFormElementsAddedBetween(const Time& delete_begin,
                                                     const Time& delete_end) {
   GenericRequest2<Time, Time>* request =
-    new GenericRequest2<Time, Time>(this,
-                                    GetNextRequestHandle(),
-                                    NULL,
-                                    delete_begin,
-                                    delete_end);
-  RegisterRequest(request);
+    new GenericRequest2<Time, Time>(
+        this, wdr_mgr_->GetNextRequestHandle(), NULL, delete_begin, delete_end);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::RemoveFormElementsAddedBetweenImpl,
                     this, request));
 }
 
 void WebDataService::RemoveExpiredFormElements() {
   WebDataRequest* request =
-      new WebDataRequest(this, GetNextRequestHandle(), NULL);
-  RegisterRequest(request);
+      new WebDataRequest(this, wdr_mgr_->GetNextRequestHandle(), NULL);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::RemoveExpiredFormElementsImpl,
                     this, request));
 }
 
 void WebDataService::RemoveFormValueForElementName(
     const string16& name, const string16& value) {
   GenericRequest2<string16, string16>* request =
-      new GenericRequest2<string16, string16>(this,
-                                              GetNextRequestHandle(),
-                                              NULL,
-                                              name, value);
-  RegisterRequest(request);
+      new GenericRequest2<string16, string16>(
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, name, value);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::RemoveFormValueForElementNameImpl,
                     this, request));
 }
 
 void WebDataService::AddAutofillProfile(const AutofillProfile& profile) {
   GenericRequest<AutofillProfile>* request =
       new GenericRequest<AutofillProfile>(
-          this, GetNextRequestHandle(), NULL, profile);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, profile);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::AddAutofillProfileImpl, this, request));
 }
 
 void WebDataService::UpdateAutofillProfile(const AutofillProfile& profile) {
   GenericRequest<AutofillProfile>* request =
       new GenericRequest<AutofillProfile>(
-          this, GetNextRequestHandle(), NULL, profile);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, profile);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::UpdateAutofillProfileImpl, this, request));
 }
 
 void WebDataService::RemoveAutofillProfile(const std::string& guid) {
   GenericRequest<std::string>* request =
       new GenericRequest<std::string>(
-          this, GetNextRequestHandle(), NULL, guid);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, guid);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::RemoveAutofillProfileImpl, this, request));
 }
 
 WebDataService::Handle WebDataService::GetAutofillProfiles(
     WebDataServiceConsumer* consumer) {
   WebDataRequest* request =
-      new WebDataRequest(this, GetNextRequestHandle(), consumer);
-  RegisterRequest(request);
+      new WebDataRequest(this, wdr_mgr_->GetNextRequestHandle(), consumer);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::GetAutofillProfilesImpl, this, request));
   return request->GetHandle();
 }
 
 void WebDataService::EmptyMigrationTrash(bool notify_sync) {
   GenericRequest<bool>* request =
       new GenericRequest<bool>(
-          this, GetNextRequestHandle(), NULL, notify_sync);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, notify_sync);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::EmptyMigrationTrashImpl, this, request));
 }
 
 void WebDataService::AddCreditCard(const CreditCard& credit_card) {
   GenericRequest<CreditCard>* request =
       new GenericRequest<CreditCard>(
-          this, GetNextRequestHandle(), NULL, credit_card);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, credit_card);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::AddCreditCardImpl, this, request));
 }
 
 void WebDataService::UpdateCreditCard(const CreditCard& credit_card) {
   GenericRequest<CreditCard>* request =
       new GenericRequest<CreditCard>(
-          this, GetNextRequestHandle(), NULL, credit_card);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, credit_card);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::UpdateCreditCardImpl, this, request));
 }
 
 void WebDataService::RemoveCreditCard(const std::string& guid) {
   GenericRequest<std::string>* request =
       new GenericRequest<std::string>(
-          this, GetNextRequestHandle(), NULL, guid);
-  RegisterRequest(request);
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, guid);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::RemoveCreditCardImpl, this, request));
 }
 
 WebDataService::Handle WebDataService::GetCreditCards(
     WebDataServiceConsumer* consumer) {
   WebDataRequest* request =
-      new WebDataRequest(this, GetNextRequestHandle(), consumer);
-  RegisterRequest(request);
+      new WebDataRequest(this, wdr_mgr_->GetNextRequestHandle(), consumer);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::GetCreditCardsImpl, this, request));
   return request->GetHandle();
 }
 
 void WebDataService::RemoveAutofillProfilesAndCreditCardsModifiedBetween(
     const Time& delete_begin,
     const Time& delete_end) {
   GenericRequest2<Time, Time>* request =
-      new GenericRequest2<Time, Time>(this,
-                                      GetNextRequestHandle(),
-                                      NULL,
-                                      delete_begin,
-                                      delete_end);
-  RegisterRequest(request);
+      new GenericRequest2<Time, Time>(
+          this, wdr_mgr_->GetNextRequestHandle(), NULL, delete_begin,
+          delete_end);
+  wdr_mgr_->RegisterRequest(request);
   ScheduleTask(FROM_HERE, Bind(
       &WebDataService::RemoveAutofillProfilesAndCreditCardsModifiedBetweenImpl,
       this, request));
 }
 
 WebDataService::~WebDataService() {
   if (is_running_ && db_) {
     DLOG_ASSERT("WebDataService dtor called without Shutdown");
     NOTREACHED();
   }
@@ skipping 12 matching lines @@
     AutofillCountry::ApplicationLocale();
 
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::InitializeDatabaseIfNecessary, this));
   ScheduleTask(FROM_HERE,
                Bind(&WebDataService::InitializeSyncableServices, this));
   return true;
 }
 
 void WebDataService::RequestCompleted(Handle h) {
-  pending_lock_.Acquire();
-  RequestMap::iterator i = pending_requests_.find(h);
-  if (i == pending_requests_.end()) {
-    NOTREACHED() << "Request completed called for an unknown request";
-    pending_lock_.Release();
-    return;
-  }
-
-  // Take ownership of the request object and remove it from the map.
-  scoped_ptr<WebDataRequest> request(i->second);
-  pending_requests_.erase(i);
-  pending_lock_.Release();
-
-  // Notify the consumer if needed.
-  WebDataServiceConsumer* consumer = NULL;
-  if (!request->IsCancelled(&consumer) && consumer) {
-    consumer->OnWebDataServiceRequestDone(request->GetHandle(),
-                                          request->GetResult());
-  } else {
-    // Nobody is taken ownership of the result, either because it is cancelled
-    // or there is no consumer. Destroy results that require special handling.
-    WDTypedResult const *result = request->GetResult();
-    if (result) {
-      if (result->GetType() == AUTOFILL_PROFILES_RESULT) {
-        const WDResult<std::vector<AutofillProfile*> >* r =
-            static_cast<const WDResult<std::vector<AutofillProfile*> >*>(
-                result);
-        std::vector<AutofillProfile*> profiles = r->GetValue();
-        STLDeleteElements(&profiles);
-      } else if (result->GetType() == AUTOFILL_CREDITCARDS_RESULT) {
-        const WDResult<std::vector<CreditCard*> >* r =
-            static_cast<const WDResult<std::vector<CreditCard*> >*>(result);
-
-        std::vector<CreditCard*> credit_cards = r->GetValue();
-        STLDeleteElements(&credit_cards);
-      }
-    }
-  }
-}
-
-void WebDataService::RegisterRequest(WebDataRequest* request) {
-  base::AutoLock l(pending_lock_);
-  pending_requests_[request->GetHandle()] = request;
+  wdr_mgr_->RequestCompleted(h);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 //
 // The following methods are executed in Chrome_WebDataThread.
 //
 ////////////////////////////////////////////////////////////////////////////////
 
 void WebDataService::DBInitFailed(sql::InitStatus init_status) {
   ShowProfileErrorDialog(
@@ skipping 86 matching lines @@
     NOTREACHED() << "Task scheduled after Shutdown()";
 }
 
 void WebDataService::ScheduleCommit() {
   if (should_commit_ == false) {
     should_commit_ = true;
     ScheduleTask(FROM_HERE, Bind(&WebDataService::Commit, this));
   }
 }
 
-int WebDataService::GetNextRequestHandle() {
-  base::AutoLock l(pending_lock_);
-  return ++next_request_handle_;
-}
-
 ////////////////////////////////////////////////////////////////////////////////
 //
 // Keywords implementation.
 //
 ////////////////////////////////////////////////////////////////////////////////
 
 void WebDataService::AddKeywordImpl(GenericRequest<TemplateURLData>* request) {
   InitializeDatabaseIfNecessary();
   if (db_ && !request->IsCancelled(NULL)) {
     db_->GetKeywordTable()->AddKeyword(request->arg());
@@ skipping 690 matching lines @@
   return autofill_profile_syncable_service_;
 }
 
 AutocompleteSyncableService* WebDataService::GetAutocompleteSyncableService()
     const {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
   DCHECK(autocomplete_syncable_service_);  // Make sure we're initialized.
 
   return autocomplete_syncable_service_;
 }
-
-
-////////////////////////////////////////////////////////////////////////////////
-//
-// WebDataRequest implementation.
-//
-////////////////////////////////////////////////////////////////////////////////
-
-WebDataService::WebDataRequest::WebDataRequest(WebDataService* service,
-                                               Handle handle,
-                                               WebDataServiceConsumer* consumer)
-    : service_(service),
-      handle_(handle),
-      cancelled_(false),
-      consumer_(consumer),
-      result_(NULL) {
-  message_loop_ = MessageLoop::current();
-}
-
-WebDataService::WebDataRequest::~WebDataRequest() {
-  delete result_;
-}
-
-WebDataService::Handle WebDataService::WebDataRequest::GetHandle() const {
-  return handle_;
-}
-
-bool WebDataService::WebDataRequest::IsCancelled(
-    WebDataServiceConsumer** consumer) const {
-  base::AutoLock l(cancel_lock_);
-  if (consumer)
-    *consumer = consumer_;
-  return cancelled_;
-}
-
-void WebDataService::WebDataRequest::Cancel() {
-  base::AutoLock l(cancel_lock_);
-  cancelled_ = true;
-  consumer_ = NULL;
-}
-
-void WebDataService::WebDataRequest::SetResult(WDTypedResult* r) {
-  result_ = r;
-}
-
-const WDTypedResult* WebDataService::WebDataRequest::GetResult() const {
-  return result_;
-}
-
-void WebDataService::WebDataRequest::RequestComplete() {
-  message_loop_->PostTask(FROM_HERE, Bind(&WebDataService::RequestCompleted,
-                                          service_.get(), handle_));
-}