[Sync] Remove UIDataTypeController

chrome/browser/sync/glue/extension_setting_data_type_controller.h

Index: chrome/browser/sync/glue/extension_setting_data_type_controller.h
diff --git a/chrome/browser/sync/glue/extension_setting_data_type_controller.h b/chrome/browser/sync/glue/extension_setting_data_type_controller.h
index ecc8ba5994bfbb613b229c140174ff3db6115090..25a343aaf7c9c326c68ca2a5d67224fae6f004d2 100644
--- a/chrome/browser/sync/glue/extension_setting_data_type_controller.h
+++ b/chrome/browser/sync/glue/extension_setting_data_type_controller.h
@@ -28,19 +28,16 @@ class ExtensionSettingDataTypeController
  public:
   // |type| is either EXTENSION_SETTINGS or APP_SETTINGS.
   // |dump_stack| is called when an unrecoverable error occurs.
-  ExtensionSettingDataTypeController(syncer::ModelType type,
-                                     const base::Closure& dump_stack,
-                                     syncer::SyncClient* sync_client,
-                                     Profile* profile);
+  ExtensionSettingDataTypeController(
+      syncer::ModelType type,
+      const base::Closure& dump_stack,
+      syncer::SyncClient* sync_client,
+      Profile* profile,
+      const scoped_refptr<base::SequencedTaskRunner>& file_thread);

[maxbogue, 2016/11/18 19:31:03]

I recently learned that it is now preferable to pass ref-counted things by value
(not const-ref), and then use std::move(). See
https://chromium.googlesource.com/chromium/src/+/master/styleguide/c++/c++.md...

[pavely, 2016/11/19 01:11:42]

Done.

   ~ExtensionSettingDataTypeController() override;
 
-  // NonFrontendDataTypeController implementation
-  syncer::ModelSafeGroup model_safe_group() const override;
-
  private:
-  // NonFrontendDataTypeController implementation.
-  bool PostTaskOnBackendThread(const tracked_objects::Location& from_here,
-                               const base::Closure& task) override;
+  // NonUIDataTypeController implementation.
   bool StartModels() override;
 
   // Only used on the UI thread.