Migrate ArcUserDataService into ArcAuthService.

chrome/browser/chromeos/arc/arc_enterprise_reporting_service.cc

Index: chrome/browser/chromeos/arc/arc_enterprise_reporting_service.cc
diff --git a/chrome/browser/chromeos/arc/arc_enterprise_reporting_service.cc b/chrome/browser/chromeos/arc/arc_enterprise_reporting_service.cc
index 47babc73befea5aab51591e2da828ba3027f9aa3..76189479376ed29b1014ef7b7d08ddc258264e4d 100644
--- a/chrome/browser/chromeos/arc/arc_enterprise_reporting_service.cc
+++ b/chrome/browser/chromeos/arc/arc_enterprise_reporting_service.cc
@@ -38,22 +38,9 @@ void ArcEnterpriseReportingService::ReportManagementState(
 
   if (state == mojom::ManagementState::MANAGED_DO_LOST) {
     DCHECK(arc::ArcServiceManager::Get());
-    ArcServiceManager::Get()->arc_user_data_service()->RequireUserDataWiped(
-        base::Bind(&ArcEnterpriseReportingService::RestartArc,
-                   weak_ptr_factory_.GetWeakPtr()));
-    ArcAuthService::Get()->StopArc();
+    ArcAuthService::Get()->RemoveArcData();

[khmel, 2016/08/04 23:23:29]

nit. May we have one function instead? They are both public and as I understand
should be used always together in right order. 

For example OnManagementLost() 

[hidehiko, 2016/08/09 13:23:56]

I'd like to keep the concept as is for now, so let me keep the "device lost"
concept here.

I'm open for better approaches/code restructuring, but let's discuss it
separately.

[khmel, 2016/08/09 15:23:11]

Acknowledged.

+    ArcAuthService::Get()->ReenableArc();
   }
 }
 
-void ArcEnterpriseReportingService::RestartArc(bool result) {
-  DCHECK(thread_checker_.CalledOnValidThread());
-  if (!result)
-    LOG(ERROR) << "Required ARC user data wipe failed.";
-
-  // Restart ARC anyway. Let the enterprise reporting instance decide whether
-  // the ARC user data wipe is still required or not.
-  VLOG(1) << "Restart ARC";
-  ArcAuthService::Get()->EnableArc();
-}
-
 }  // namespace arc