Profile shouldn't own PersonalDataManager

chrome/browser/webdata/web_data_service.cc

 // Copyright (c) 2011 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/message_loop.h"
 #include "base/stl_util.h"
 #include "base/task.h"
 #include "base/threading/thread.h"
@@ skipping 24 matching lines @@
 ////////////////////////////////////////////////////////////////////////////////
 //
 // WebDataService implementation.
 //
 ////////////////////////////////////////////////////////////////////////////////
 
 using base::Time;
 using webkit_glue::FormField;
 using webkit_glue::PasswordForm;
 
+namespace {
+// A task used by WebDataService (for Sync mainly) to inform the
+// PersonalDataManager living on the UI thread that it needs to refresh.
+class NotifyOfMultipleAutofillChangesTask : public Task {
+ public:
+  explicit NotifyOfMultipleAutofillChangesTask(
+      WebDataService* web_data_service);
+  virtual ~NotifyOfMultipleAutofillChangesTask();
+  virtual void Run();
+ private:
+  WebDataService* web_data_service_;
+};
+
+NotifyOfMultipleAutofillChangesTask::NotifyOfMultipleAutofillChangesTask(
+    WebDataService* web_data_service) : web_data_service_(web_data_service) {
+}
+
+NotifyOfMultipleAutofillChangesTask::~NotifyOfMultipleAutofillChangesTask() {}
+
+void NotifyOfMultipleAutofillChangesTask::Run() {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+
+  NotificationService::current()->Notify(
+      chrome::NOTIFICATION_AUTOFILL_MULTIPLE_CHANGED,
+      Source<WebDataService>(web_data_service_),
+      NotificationService::NoDetails());
+}
+
+}  // namespace
+
 WDAppImagesResult::WDAppImagesResult() : has_all_images(false) {}
 
 WDAppImagesResult::~WDAppImagesResult() {}
 
 WDKeywordsResult::WDKeywordsResult()
   : default_search_provider_id(0),
     builtin_keyword_version(0) {
 }
 
 WDKeywordsResult::~WDKeywordsResult() {}
@@ skipping 497 matching lines @@
                                       NULL,
                                       delete_begin,
                                       delete_end);
   RegisterRequest(request);
   ScheduleTask(NewRunnableMethod(
       this,
       &WebDataService::RemoveAutofillProfilesAndCreditCardsModifiedBetweenImpl,
       request));
 }
 
+void WebDataService::NotifyOfMultipleAutofillChanges() {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
+  BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
+      new NotifyOfMultipleAutofillChangesTask(this));
+}
+
 WebDataService::~WebDataService() {
   if (is_running_ && db_) {
     DLOG_ASSERT("WebDataService dtor called without Shutdown");
   }
 }
 
 bool WebDataService::InitWithPath(const FilePath& path) {
   path_ = path;
   is_running_ = true;
   ScheduleTask(NewRunnableMethod(this,
@@ skipping 867 matching lines @@
   return result_;
 }
 
 void WebDataService::WebDataRequest::RequestComplete() {
   WebDataService* s = service_;
   Task* t = NewRunnableMethod(s,
                               &WebDataService::RequestCompleted,
                               handle_);
   message_loop_->PostTask(FROM_HERE, t);
 }