Revert of [Sync] Make directory types registration explicit in ModelTypeRegistry

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 d05bddc569b914cb67f4690b81554d9a0591b456..b4b2777894bfe0a1917aba93752286f1a84be48e 100644
--- a/components/sync/driver/directory_data_type_controller.h
+++ b/components/sync/driver/directory_data_type_controller.h
@@ -23,13 +23,12 @@
 
   // DataTypeController implementation.
   bool ShouldLoadModelBeforeConfigure() const override;
+  void GetAllNodes(const AllNodesCallback& callback) override;
+  void GetStatusCounters(const StatusCountersCallback& callback) override;
 
-  // Directory types need to register with sync engine before LoadModels because
-  // downloading initial data happens in parallel with LoadModels.
-  void BeforeLoadModels(ModelTypeConfigurer* configurer) override;
-
-  // Directory based data types register with backend before LoadModels in
-  // BeforeLoadModels. No need to do anything in RegisterWithBackend.
+  // Directory based data types don't need to register with backend.
+  // ModelTypeRegistry will create all necessary objects in
+  // SetEnabledDirectoryTypes based on routing info.
   void RegisterWithBackend(base::Callback<void(bool)> set_downloaded,
                            ModelTypeConfigurer* configurer) override;
 
@@ -46,9 +45,6 @@
   // the data type's ChangeProcessor registration with the backend).
   // See ModelTypeConfigurer::DeactivateDataType for more details.
   void DeactivateDataType(ModelTypeConfigurer* configurer) override;
-
-  void GetAllNodes(const AllNodesCallback& callback) override;
-  void GetStatusCounters(const StatusCountersCallback& callback) override;
 
   // Returns a ListValue representing all nodes for a specified type by querying
   // the directory.