Ensure objects recorded using Undo.RecordObject or Undo.RecordObjects are registered as an undoable action. In most cases there is no reason to invoke Undo.FlushUndoRecordObjects since it's automatically done right after mouse-up and certain other events that conventionally marks the end of an action.

public static void FlushUndoRecordObjects()