sync: only use RecordUnrecoverableError for legacy types.

components/sync_driver/data_type_controller.h

Index: components/sync_driver/data_type_controller.h
diff --git a/components/sync_driver/data_type_controller.h b/components/sync_driver/data_type_controller.h
index 37fefc6bb330cf4f617b67919119871d593067c5..84faf9aaa49cae270a4f8c808d87655e4a9ed0ef 100644
--- a/components/sync_driver/data_type_controller.h
+++ b/components/sync_driver/data_type_controller.h
@@ -147,20 +147,14 @@ class DataTypeController
 
   virtual ~DataTypeController();
 
-  // Handles the reporting of unrecoverable error. It records stuff in
-  // UMA and reports to breakpad.
-  // Virtual for testing purpose.
-  virtual void RecordUnrecoverableError(
-      const tracked_objects::Location& from_here,
-      const std::string& message);
-
   syncer::UserShare* user_share() const;
 
- private:
-  syncer::UserShare* user_share_;
-
   // The callback that will be invoked when an unrecoverable error occurs.
+  // TODO(sync): protected for use by legacy controllers.
   base::Closure error_callback_;
+
+ private:
+  syncer::UserShare* user_share_;
 };
 
 }  // namespace browser_sync