[Sync] Remove UIDataTypeController

chrome/browser/supervised_user/supervised_user_sync_data_type_controller.h

Index: chrome/browser/supervised_user/supervised_user_sync_data_type_controller.h
diff --git a/chrome/browser/supervised_user/supervised_user_sync_data_type_controller.h b/chrome/browser/supervised_user/supervised_user_sync_data_type_controller.h
index 4d40e334434808183e3acab49920e88dccde582c..f3a972de67ec622612dd056b530fc9d30c08c538 100644
--- a/chrome/browser/supervised_user/supervised_user_sync_data_type_controller.h
+++ b/chrome/browser/supervised_user/supervised_user_sync_data_type_controller.h
@@ -7,7 +7,7 @@
 
 #include "base/macros.h"
 #include "components/sync/driver/data_type_controller.h"
-#include "components/sync/driver/ui_data_type_controller.h"
+#include "components/sync/driver/non_ui_data_type_controller.h"
 
 class Profile;
 
@@ -15,10 +15,10 @@ namespace syncer {
 class SyncClient;
 }
 
-// A UIDataTypeController for supervised user sync datatypes, which enables or
+// A DataTypeController for supervised user sync datatypes, which enables or
 // disables these types based on the profile's IsSupervised state.
 class SupervisedUserSyncDataTypeController
-    : public syncer::UIDataTypeController {
+    : public syncer::NonUIDataTypeController {

[Marc Treib, 2016/11/22 09:05:59]

Should NonUIDataTypeController be renamed now? IMO the name is misleading if
it's now used for both UI and non-UI thread controllers.

[pavely, 2016/11/22 19:17:52]

I failed to remove all references to UIDataTypeController, switching Sessions to
NonUI caused integration test failures which I'm going to address in separate
CL. I'll rename it once I'm done.

  public:
   // |dump_stack| is called when an unrecoverable error occurs.
   SupervisedUserSyncDataTypeController(syncer::ModelType type,
@@ -27,6 +27,7 @@ class SupervisedUserSyncDataTypeController
                                        Profile* profile);
   ~SupervisedUserSyncDataTypeController() override;
 
+  // NonUIDataTypeController implementation.
   bool ReadyForStart() const override;
 
  private: