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

Unified Diff: chrome/browser/sync/glue/autofill_data_type_controller.h

Issue 15927029: Replace WebDatabaseObserver with callbacks (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 7 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/sync/glue/autofill_data_type_controller.h
diff --git a/chrome/browser/sync/glue/autofill_data_type_controller.h b/chrome/browser/sync/glue/autofill_data_type_controller.h
index 7fc275fcd62815efe79e6d94070576863aa66457..bb513d3f48612140a91d5a62d4384943577de816 100644
--- a/chrome/browser/sync/glue/autofill_data_type_controller.h
+++ b/chrome/browser/sync/glue/autofill_data_type_controller.h
@@ -11,7 +11,6 @@
#include "base/gtest_prod_util.h"
#include "base/memory/ref_counted.h"
#include "chrome/browser/sync/glue/non_ui_data_type_controller.h"
-#include "components/webdata/common/web_database_observer.h"
namespace autofill {
class AutofillWebDataService;
@@ -21,8 +20,7 @@ namespace browser_sync {
// A class that manages the startup and shutdown of autofill sync.
class AutofillDataTypeController
- : public NonUIDataTypeController,
- public WebDatabaseObserver {
+ : public NonUIDataTypeController {
public:
AutofillDataTypeController(
ProfileSyncComponentsFactory* profile_sync_factory,
@@ -37,9 +35,6 @@ class AutofillDataTypeController
// 163431 is addressed / implemented.
virtual void StartAssociating(const StartCallback& start_callback) OVERRIDE;
- // WebDatabaseObserver implementation.
- virtual void WebDatabaseLoaded() OVERRIDE;
-
protected:
virtual ~AutofillDataTypeController();
@@ -57,9 +52,11 @@ class AutofillDataTypeController
// Self-invoked on the DB thread to call the AutocompleteSyncableService with
// an updated value of autofill culling settings.
- void UpdateAutofillCullingSettings(bool cull_expired_entries);
+ void UpdateAutofillCullingSettings(bool cull_expired_entries,
+ scoped_refptr<autofill::AutofillWebDataService> web_data_service);
- scoped_refptr<autofill::AutofillWebDataService> web_data_service_;
+ // Callback once WebDatabase has loaded.
+ void WebDatabaseLoaded();
DISALLOW_COPY_AND_ASSIGN(AutofillDataTypeController);
};

Powered by Google App Engine
This is Rietveld 408576698