Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1602)

Unified Diff: chrome/browser/sync/glue/data_type_manager.cc

Issue 7453014: [Sync] Refactor sync datatype error handling. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 9 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..65b285a030bcd227ddd9897931a79ae9517e7f7d 100644
--- a/chrome/browser/sync/glue/data_type_manager.cc
+++ b/chrome/browser/sync/glue/data_type_manager.cc
@@ -6,12 +6,25 @@
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);
}
} // namespace browser_sync

Powered by Google App Engine
This is Rietveld 408576698