[Sync] Move DataTypeErrorHandler to //sync.

components/sync_driver/change_processor.h

Index: components/sync_driver/change_processor.h
diff --git a/components/sync_driver/change_processor.h b/components/sync_driver/change_processor.h
index ef0073ef038cc6c13930d6b14b41798c9efb3ef0..d6a6760d732f67b651af0f150edc4423f81bb3b6 100644
--- a/components/sync_driver/change_processor.h
+++ b/components/sync_driver/change_processor.h
@@ -8,12 +8,13 @@
 #include <stdint.h>
 
 #include "base/macros.h"
-#include "components/sync_driver/data_type_error_handler.h"
 #include "sync/internal_api/public/base_transaction.h"
 #include "sync/internal_api/public/change_record.h"
+#include "sync/internal_api/public/data_type_error_handler.h"

[pavely, 2016/05/11 18:29:04]

You don't need this include, you already forward declared the class. 

[maxbogue, 2016/05/11 19:03:02]

Done.

 #include "sync/internal_api/public/user_share.h"
 
 namespace syncer {
+class DataTypeErrorHandler;
 class UnrecoverableErrorHandler;
 }  // namespace syncer
 
@@ -25,7 +26,7 @@ class ModelAssociator;
 // native model.  This does not currently distinguish between model data types.
 class ChangeProcessor {
  public:
-  explicit ChangeProcessor(DataTypeErrorHandler* error_handler);
+  explicit ChangeProcessor(syncer::DataTypeErrorHandler* error_handler);
   virtual ~ChangeProcessor();
 
   // Call when the processor should accept changes from either provided model
@@ -56,11 +57,11 @@ class ChangeProcessor {
   // implementation-specific work.
   virtual void StartImpl() = 0;
 
-  DataTypeErrorHandler* error_handler() const;
+  syncer::DataTypeErrorHandler* error_handler() const;
   virtual syncer::UserShare* share_handle() const;
 
  private:
-  DataTypeErrorHandler* error_handler_;  // Guaranteed to outlive us.
+  syncer::DataTypeErrorHandler* error_handler_;  // Guaranteed to outlive us.
 
   // The sync model we are processing changes from.
   syncer::UserShare* share_handle_;