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

components/sync/BUILD.gn

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/buildflag_header.gni")
 import("//build/config/features.gni")
 import("//components/sync/protocol/protocol_sources.gni")
 import("//testing/test.gni")
 
 static_library("sync") {
   sources = [
     "api/attachments/attachment.cc",
     "api/attachments/attachment.h",
     "api/attachments/attachment_id.cc",
     "api/attachments/attachment_id.h",
     "api/attachments/attachment_metadata.cc",
     "api/attachments/attachment_metadata.h",
     "api/attachments/attachment_store.cc",
     "api/attachments/attachment_store.h",
     "api/attachments/attachment_store_backend.cc",
     "api/attachments/attachment_store_backend.h",
     "api/conflict_resolution.cc",
     "api/conflict_resolution.h",
     "api/data_batch.h",
+    "api/data_type_error_handler.h",
+    "api/data_type_error_handler_impl.cc",
+    "api/data_type_error_handler_impl.h",
     "api/entity_change.cc",
     "api/entity_change.h",
     "api/entity_data.cc",
     "api/entity_data.h",
     "api/metadata_batch.cc",
     "api/metadata_batch.h",
     "api/metadata_change_list.h",
     "api/model_type_change_processor.cc",
     "api/model_type_change_processor.h",
     "api/model_type_service.cc",
@@ skipping 93 matching lines @@
     "core/base_transaction.h",
     "core/change_record.cc",
     "core/change_record.h",
     "core/configure_reason.h",
     "core/data_batch_impl.cc",
     "core/data_batch_impl.h",
     "core/data_type_association_stats.cc",
     "core/data_type_association_stats.h",
     "core/data_type_debug_info_listener.cc",
     "core/data_type_debug_info_listener.h",
-    "core/data_type_error_handler.h",
     "core/delete_journal.cc",
     "core/delete_journal.h",
     "core/http_bridge.cc",
     "core/http_bridge.h",
     "core/http_bridge_network_resources.cc",
     "core/http_bridge_network_resources.h",
     "core/http_post_provider_factory.h",
     "core/http_post_provider_interface.h",
     "core/internal_components_factory.h",
     "core/internal_components_factory_impl.cc",
@@ skipping 515 matching lines @@
     "//testing/gtest",
   ]
 
   defines = [ "SYNC_TEST" ]
   configs += [ "//build/config/compiler:wexit_time_destructors" ]
 }
 
 static_library("test_support_sync_core_impl") {
   testonly = true
   sources = [
-    "core/test/data_type_error_handler_mock.h",
+    "api/data_type_error_handler_mock.cc",
+    "api/data_type_error_handler_mock.h",
     "core/test/fake_model_type_connector.h",
     "core/test/fake_model_type_processor.h",
     "core/test/fake_sync_manager.h",
     "core/test/model_type_store_test_util.h",
     "core/test/sync_manager_factory_for_profile_sync_test.h",
     "core/test/test_entry_factory.h",
     "core/test/test_internal_components_factory.h",
     "core/test/test_user_share.h",
-    "core_impl/test/data_type_error_handler_mock.cc",
     "core_impl/test/fake_model_type_connector.cc",
     "core_impl/test/fake_model_type_processor.cc",
     "core_impl/test/fake_sync_manager.cc",
     "core_impl/test/model_type_store_test_util.cc",
     "core_impl/test/sync_manager_factory_for_profile_sync_test.cc",
     "core_impl/test/sync_manager_for_profile_sync_test.cc",
     "core_impl/test/sync_manager_for_profile_sync_test.h",
     "core_impl/test/test_entry_factory.cc",
     "core_impl/test/test_internal_components_factory.cc",
     "core_impl/test/test_user_share.cc",
@@ skipping 402 matching lines @@
     deps = [
       ":fake_server_jni",
       ":sync",
       ":test_support_sync_fake_server",
       "//base",
       "//testing/gtest",
       "//url:url",
     ]
   }
 }