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

Unified Diff: components/autofill/core/browser/webdata/autofill_data_type_controller.cc

Issue 2511703004: [Sync] Remove UIDataTypeController (Closed)
Patch Set: Fix integration test issue. Created 4 years, 1 month 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: components/autofill/core/browser/webdata/autofill_data_type_controller.cc
diff --git a/components/autofill/core/browser/webdata/autofill_data_type_controller.cc b/components/autofill/core/browser/webdata/autofill_data_type_controller.cc
index d1399bfc3a2acfc5f6b4f5f5d24b1f8c0bfab1d1..ba76a2b872c7a06a1241d5a148afad93b0736c43 100644
--- a/components/autofill/core/browser/webdata/autofill_data_type_controller.cc
+++ b/components/autofill/core/browser/webdata/autofill_data_type_controller.cc
@@ -4,6 +4,8 @@
#include "components/autofill/core/browser/webdata/autofill_data_type_controller.h"
+#include <utility>
+
#include "base/bind.h"
#include "base/metrics/histogram.h"
#include "components/autofill/core/browser/webdata/autocomplete_syncable_service.h"
@@ -14,18 +16,17 @@
namespace browser_sync {
AutofillDataTypeController::AutofillDataTypeController(
- const scoped_refptr<base::SingleThreadTaskRunner>& db_thread,
+ scoped_refptr<base::SingleThreadTaskRunner> db_thread,
const base::Closure& dump_stack,
syncer::SyncClient* sync_client,
const scoped_refptr<autofill::AutofillWebDataService>& web_data_service)
- : NonUIDataTypeController(syncer::AUTOFILL, dump_stack, sync_client),
- db_thread_(db_thread),
+ : NonUIDataTypeController(syncer::AUTOFILL,
+ dump_stack,
+ sync_client,
+ syncer::GROUP_DB,
+ std::move(db_thread)),
web_data_service_(web_data_service) {}
-syncer::ModelSafeGroup AutofillDataTypeController::model_safe_group() const {
- return syncer::GROUP_DB;
-}
-
void AutofillDataTypeController::WebDatabaseLoaded() {
DCHECK(CalledOnValidThread());
DCHECK_EQ(MODEL_STARTING, state());
@@ -37,13 +38,6 @@ AutofillDataTypeController::~AutofillDataTypeController() {
DCHECK(CalledOnValidThread());
}
-bool AutofillDataTypeController::PostTaskOnBackendThread(
- const tracked_objects::Location& from_here,
- const base::Closure& task) {
- DCHECK(CalledOnValidThread());
- return db_thread_->PostTask(from_here, task);
-}
-
bool AutofillDataTypeController::StartModels() {
DCHECK(CalledOnValidThread());
DCHECK_EQ(MODEL_STARTING, state());

Powered by Google App Engine
This is Rietveld 408576698