[Sync] Refactor sync datatype error handling.

chrome/browser/sync/glue/data_type_manager.cc

Index: chrome/browser/sync/glue/data_type_manager.cc
diff --git a/chrome/browser/sync/glue/data_type_manager.cc b/chrome/browser/sync/glue/data_type_manager.cc
index 45b836efec4ad85621482b634a7250e0480205ce..186205abf29820fe702e78f1cc8978d460e74283 100644
--- a/chrome/browser/sync/glue/data_type_manager.cc
+++ b/chrome/browser/sync/glue/data_type_manager.cc
@@ -6,12 +6,42 @@
 
 namespace browser_sync {
 
-DataTypeManager::ConfigureResultWithErrorLocation::
-    ~ConfigureResultWithErrorLocation() {}
+DataTypeManager::ConfigureResult::ConfigureResult() {}
 
-DataTypeManager::ConfigureResultWithErrorLocation::
-    ConfigureResultWithErrorLocation()
-    : result(OK) {
+DataTypeManager::ConfigureResult::ConfigureResult(ConfigureStatus status,
+                                                  TypeSet requested_types)
+    : status(status),
+      requested_types(requested_types) {
+  DCHECK_EQ(OK, status);
+}
+
+DataTypeManager::ConfigureResult::ConfigureResult(
+    ConfigureStatus status,
+    TypeSet requested_types,
+    TypeSet failed_types,
+    const tracked_objects::Location& location)
+    : status(status),
+      requested_types(requested_types),
+      failed_types(failed_types),
+      location(location) {
+  DCHECK_NE(OK, status);
+}
+
+// Static.
+std::string DataTypeManager::ConfigureStatusToString(ConfigureStatus status) {
+  switch (status) {
+    case OK:
+      return "Ok";
+    case ASSOCIATION_FAILED:
+      return "Association Failed";
+    case ABORTED:
+      return "Aborted";
+    case UNRECOVERABLE_ERROR:
+      return "Unrecoverable Error";
+    default:
+      NOTREACHED();
+      return std::string();
+  }
 }
 
 }  // namespace browser_sync