| Index: chrome/browser/sync/engine/cleanup_disabled_types_command.cc
|
| diff --git a/chrome/browser/sync/engine/cleanup_disabled_types_command.cc b/chrome/browser/sync/engine/cleanup_disabled_types_command.cc
|
| index 49a0fda04acb3a7b7df7bf86a0bdaad27c7f460f..b1de7f5c076a088400726a19147e0c9d1c5c8da3 100644
|
| --- a/chrome/browser/sync/engine/cleanup_disabled_types_command.cc
|
| +++ b/chrome/browser/sync/engine/cleanup_disabled_types_command.cc
|
| @@ -17,10 +17,10 @@ namespace browser_sync {
|
| CleanupDisabledTypesCommand::CleanupDisabledTypesCommand() {}
|
| CleanupDisabledTypesCommand::~CleanupDisabledTypesCommand() {}
|
|
|
| -void CleanupDisabledTypesCommand::ExecuteImpl(sessions::SyncSession* session) {
|
| +SyncerError CleanupDisabledTypesCommand::ExecuteImpl(
|
| + sessions::SyncSession* session) {
|
| using syncable::ModelTypeSet;
|
| using syncable::ModelTypeSetToString;
|
| -
|
| // Because a full directory purge is slow, we avoid purging
|
| // undesired types unless we have reason to believe they were
|
| // previously enabled. Because purging could theoretically fail on
|
| @@ -61,16 +61,17 @@ void CleanupDisabledTypesCommand::ExecuteImpl(sessions::SyncSession* session) {
|
| << ", to_cleanup = " << ModelTypeSetToString(to_cleanup);
|
|
|
| if (to_cleanup.Empty())
|
| - return;
|
| + return NO_ERROR;
|
|
|
| syncable::ScopedDirLookup dir(session->context()->directory_manager(),
|
| session->context()->account_name());
|
| if (!dir.good()) {
|
| LOG(ERROR) << "Scoped dir lookup failed!";
|
| - return;
|
| + return DIRECTORY_LOOKUP_FAILED;
|
| }
|
|
|
| dir->PurgeEntriesWithTypeIn(to_cleanup);
|
| + return NO_ERROR;
|
| }
|
|
|
| } // namespace browser_sync
|
|
|