Add return values to SyncerCommand

chrome/browser/sync/engine/cleanup_disabled_types_command.cc

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