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 |