[Sync] Convert DTCs to be not RefCounted and NonThreadSafe.

components/sync/driver/directory_data_type_controller.h

Index: components/sync/driver/directory_data_type_controller.h
diff --git a/components/sync/driver/directory_data_type_controller.h b/components/sync/driver/directory_data_type_controller.h
index 16d1f94412ccb8aac78c5125049d749563954fad..fb0a45b0df4e1ca5421a313987d0f4755ea03a98 100644
--- a/components/sync/driver/directory_data_type_controller.h
+++ b/components/sync/driver/directory_data_type_controller.h
@@ -17,6 +17,8 @@ class ChangeProcessor;
 // Base class for Directory based Data type controllers.
 class DirectoryDataTypeController : public DataTypeController {
  public:
+  ~DirectoryDataTypeController() override;
+
   // DataTypeController implementation.
   bool ShouldLoadModelBeforeConfigure() const override;
   void GetAllNodes(const AllNodesCallback& callback) override;
@@ -47,6 +49,10 @@ class DirectoryDataTypeController : public DataTypeController {
       syncer::syncable::Directory* directory);
 
  protected:
+  DirectoryDataTypeController(syncer::ModelType type,
+                              const base::Closure& dump_stack,
+                              SyncClient* sync_client);
+
   // The model safe group of this data type.  This should reflect the
   // thread that should be used to modify the data type's native
   // model.
@@ -56,13 +62,6 @@ class DirectoryDataTypeController : public DataTypeController {
   // Returns NULL if the ChangeProcessor isn't created or connected.
   virtual ChangeProcessor* GetChangeProcessor() const = 0;
 
-  DirectoryDataTypeController(
-      const scoped_refptr<base::SingleThreadTaskRunner>& ui_thread,
-      const base::Closure& error_callback,
-      SyncClient* sync_client);
-
-  ~DirectoryDataTypeController() override;
-
   SyncClient* const sync_client_;
 };