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

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

Issue 15701022: [Sync] Add support for sync Persistence Errors (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move bookmark change into separate patch Created 7 years, 6 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_impl.cc
diff --git a/chrome/browser/sync/glue/data_type_manager_impl.cc b/chrome/browser/sync/glue/data_type_manager_impl.cc
index 01f8fcb4fbd7aa88066f50c7ba715558adbe12b2..7bcfd43382dfe827ab32eeba0670f602ead23c5a 100644
--- a/chrome/browser/sync/glue/data_type_manager_impl.cc
+++ b/chrome/browser/sync/glue/data_type_manager_impl.cc
@@ -35,9 +35,11 @@ GenerateCryptoErrorsForTypes(syncer::ModelTypeSet encrypted_types) {
FailedDataTypesHandler::TypeErrorMap crypto_errors;
for (syncer::ModelTypeSet::Iterator iter = encrypted_types.First();
iter.Good(); iter.Inc()) {
- crypto_errors[iter.Get()] = syncer::SyncError(FROM_HERE,
- "Cryptographer not ready.",
- iter.Get());
+ crypto_errors[iter.Get()] = syncer::SyncError(
+ FROM_HERE,
+ syncer::SyncError::CRYPTO_ERROR,
+ "",
+ iter.Get());
}
return crypto_errors;
}
@@ -187,9 +189,7 @@ void DataTypeManagerImpl::Restart(syncer::ConfigureReason reason) {
failed_data_types_handler_->GetCryptoErrorTypes());
FailedDataTypesHandler::TypeErrorMap crypto_errors =
GenerateCryptoErrorsForTypes(encrypted_types);
- failed_data_types_handler_->UpdateFailedDataTypes(
- crypto_errors,
- FailedDataTypesHandler::CRYPTO);
+ failed_data_types_handler_->UpdateFailedDataTypes(crypto_errors);
} else {
failed_data_types_handler_->ResetCryptoErrors();
}
@@ -298,7 +298,9 @@ void DataTypeManagerImpl::DownloadReady(
std::string error_msg =
"Configuration failed for types " +
syncer::ModelTypeSetToString(failed_configuration_types);
- syncer::SyncError error(FROM_HERE, error_msg,
+ syncer::SyncError error(FROM_HERE,
+ syncer::SyncError::UNRECOVERABLE_ERROR,
+ error_msg,
failed_configuration_types.First().Get());
Abort(UNRECOVERABLE_ERROR, error);
return;
@@ -394,15 +396,12 @@ void DataTypeManagerImpl::OnModelAssociationDone(
failed_data_types_handler_->GetCryptoErrorTypes());
FailedDataTypesHandler::TypeErrorMap crypto_errors =
GenerateCryptoErrorsForTypes(encrypted_types);
- failed_data_types_handler_->UpdateFailedDataTypes(
- crypto_errors,
- FailedDataTypesHandler::CRYPTO);
+ failed_data_types_handler_->UpdateFailedDataTypes(crypto_errors);
}
if (!result.failed_data_types.empty()) {
needs_reconfigure_ = true;
failed_data_types_handler_->UpdateFailedDataTypes(
- result.failed_data_types,
- FailedDataTypesHandler::STARTUP);
+ result.failed_data_types);
}
}
@@ -489,7 +488,7 @@ void DataTypeManagerImpl::Abort(ConfigureStatus status,
DCHECK_NE(OK, status);
std::map<syncer::ModelType, syncer::SyncError> errors;
if (error.IsSet())
- errors[error.type()] = error;
+ errors[error.model_type()] = error;
ConfigureResult result(status,
last_requested_types_,
errors,
« no previous file with comments | « chrome/browser/sync/glue/data_type_controller.cc ('k') | chrome/browser/sync/glue/failed_data_types_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698