microsoft / Windows-task-snippets

Reusable code snippets for the Universal Windows Platform.
MIT License
307 stars 66 forks source link

Cancellation in UI-thread-task-await-from-background-thread.md #13

Open sjb-sjb opened 4 years ago

sjb-sjb commented 4 years ago

The snippet "UI-thread-task-await-from-background-thread.md" does not handle cancellation. It is desirable that the state of the TaskCompletionSource reflect the cancellation state of the task that is being run in the background on the UI thread.

My first attempt at doing this was to register a cancellation callback before RunAsync:

        if (cancellationToken != CancellationToken.None) {
            cancellationToken.Register(() => taskCompletionSource.TrySetCanceled(cancellationToken));
        }

However, if a cancellation occurs then I believe it would be possible for the SetException to occur before the cancellationToken callback is executed. A better approach is to catch cancellation exceptions thrown with the correct token:

    public static async Task<TOut> RunTaskAsync<TOut>( this CoreDispatcher dispatcher, Func<Task<TOut>> actionAsync, CoreDispatcherPriority priority, CancellationToken cancellationToken)
    {
        var taskCompletionSource = new TaskCompletionSource<TOut>(TaskCreationOptions.RunContinuationsAsynchronously);
        await ThreadHelpers.Dispatcher.RunAsync(
            priority,
            async () => {
                try {
                    taskCompletionSource.TrySetResult(await actionAsync()); 
                } catch (OperationCanceledException cancellationException) when (cancellationException.CancellationToken == cancellationToken) {
                    taskCompletionSource.SetCanceled();
                } catch (Exception otherException) {
                    taskCompletionSource.SetException(otherException);
                }
            }
        );
        return await taskCompletionSource.Task;
    }