[Sync] Refactor sync datatype error handling.

chrome/browser/sync/profile_sync_service.cc

Index: chrome/browser/sync/profile_sync_service.cc
diff --git a/chrome/browser/sync/profile_sync_service.cc b/chrome/browser/sync/profile_sync_service.cc
index 0cd4727cfd6179eab7a59fb92aa22b22c20ea223..deafd133ea12637d663a160092b706f251f4f9fb 100644
--- a/chrome/browser/sync/profile_sync_service.cc
+++ b/chrome/browser/sync/profile_sync_service.cc
@@ -1254,13 +1254,12 @@ void ProfileSyncService::Observe(int type,
       break;
     }
     case chrome::NOTIFICATION_SYNC_CONFIGURE_DONE: {
-      DataTypeManager::ConfigureResultWithErrorLocation* result_with_location =
-          Details<DataTypeManager::ConfigureResultWithErrorLocation>(
-              details).ptr();
+      DataTypeManager::ConfigureResult* result =
+          Details<DataTypeManager::ConfigureResult>(details).ptr();
 
-      DataTypeManager::ConfigureResult result = result_with_location->result;
-      VLOG(1) << "PSS SYNC_CONFIGURE_DONE called with result: " << result;
-      if (result == DataTypeManager::ABORTED &&
+      DataTypeManager::ConfigureStatus status = result->status;
+      VLOG(1) << "PSS SYNC_CONFIGURE_DONE called with status: " << status;
+      if (status == DataTypeManager::ABORTED &&
           expect_sync_configuration_aborted_) {
         VLOG(0) << "ProfileSyncService::Observe Sync Configure aborted";
         expect_sync_configuration_aborted_ = false;
@@ -1268,11 +1267,14 @@ void ProfileSyncService::Observe(int type,
       }
       // Clear out the gaia password if it is already there.
       gaia_password_ = std::string();
-      if (result != DataTypeManager::OK) {
+      if (status != DataTypeManager::OK) {
         VLOG(0) << "ProfileSyncService::Observe: Unrecoverable error detected";
-        std::string message = StringPrintf("Sync Configuration failed with %d",
-                                            result);
-        OnUnrecoverableError(*(result_with_location->location), message);
+        std::string message =
+            StringPrintf(
+                "Sync Configuration failed while configuring %s with result %d",
+                syncable::ModelTypeSetToString(result->failed_types).c_str(),
+                status);
+        OnUnrecoverableError(result->location, message);
         cached_passphrase_ = CachedPassphrase();
         return;
       }