[UniRx.Async] UnityWebRequestAsyncOperationConfiguredAwaiter周辺で困った話

# CancelしてもUnityWebRequestはAbortされない

UniTaskにはUnityWebRequestをawaitできる機能が付いていて、さらにそれをキャンセルできるインターフェースが用意されている。
こんな感じ。

```cs
CancellationTokenSource cts;

public async void SendRequest()
{
    cts = new CancellationTokenSource();
    var req = UnityWebRequest.Get("http://example.com");

    await req.SendWebRequest().CongigureAwait(cts.Token);
    Debug.Log("Completed: " + req.downloadHandler.text);
}

public void Cancel()
{
    if (cts != null) cts.Cancel();
}
```

が、これでキャンセルしても**UnityWebRequestはAbortされない**。
AwaiterのGetResultでOperationCanceledExceptionがthrowされるので、await以降の処理は実行されない。（[UnityAsyncExtensions.cs#L640-L642](https://github.com/neuecc/UniRx/blob/89b406c6e88c964507b1159111aa6d357e2b7d0f/Assets/Plugins/UniRx/Scripts/Async/UnityAsyncExtensions.cs#L640-L642)）
が、UnityWebRequestはAbortされていないので、例えば以下の場合に普通に通信完了時にレスポンステキストがログに出力される。（await使うのにわざわざこう書くことは無いと思われるが）

```cs
CancellationTokenSource cts;

public async void SendRequest()
{
    cts = new CancellationTokenSource();
    var req = UnityWebRequest.Get("http://example.com");

    var asyncOp = req.SendWebRequest();
    asyncOp.completed += _ => {
        Debug.Log("UnityWebRequest completed");
        Debug.Log("Completed: " + req.downloadHandler.text);
    };
    await asyncOp.CongigureAwait(cts.Token);
    Debug.Log("After await");
}

public void Cancel()
{
    if (cts != null) cts.Cancel();
}
```

なので、**通信も終了させたい場合はcts.Cancel()と同時にUnityWebRequestもAbortする**ように気を付けておかないといけない。

わざわざ[Issue](https://github.com/neuecc/UniRx/issues/361)も投げてみたけど、よくよく考えてみればあくまで「UnityWebRequestAsyncOperation」をAwaitしてる訳で、UnityWebRequestをAwaitしてる訳ではないので、Abortされないのはむしろ自然な挙動だったと気付いた。（「通信処理の完了を待機する」のをキャンセルするので「通信処理」自体はキャンセルされないという感じ）
…という事で、仕様でいいんですよね？

# Progress.ReportがContinuationの後に実行される

どういうことかというと、以下のコードはNullReferenceExceptionを発生させる。

```cs
[SerializeField] Text _text;

public async void StartRequest()
{
    var req = UnityWebRequest.Get("http://example.com");
    await req.SendWebRequest().ConfigureAwait(new Progress<float>(x => _text.text = x.ToString()));
    Debug.Log("StartRequest Completed");
    GameObject.Destroy(_text);
    _text = null;
}
```

![image.png](https://qiita-image-store.s3.amazonaws.com/0/97014/fbc2a44e-398d-917b-e3d6-a0dc92754d29.png)

何故かというと、**await後の処理（Continuation）が行われた後にProgressのReportが実行されている**。
上記コードではawait後に\_textをDestroyしている為、その後にProgressのReportをしようとして\_textを参照してNullReferenceExceptionになるという流れ。

UniTaskの内部ではこの辺でProgressのReportとcontinuationのinvokeを行っている
→ [UnityAsyncExtensions.cs#L656-L666](https://github.com/neuecc/UniRx/blob/89b406c6e88c964507b1159111aa6d357e2b7d0f/Assets/Plugins/UniRx/Scripts/Async/UnityAsyncExtensions.cs#L656-L666)

```cs
if (progress != null)
{
    progress.Report(asyncOperation.progress);
}

if (asyncOperation.isDone)
{
    this.result = asyncOperation.webRequest;
    InvokeContinuation(AwaiterStatus.Succeeded);
    return false;
}   
```

一見するとIProgress.ReportのあとにcontinuationがInvokeされているので問題無いように見える。が、この問題の本質は**Progressクラス**そのものにある。
Progressクラスは簡単に言うと、コンストラクタでSyncronizationContextを取得し、Report時にはSyncronizationContext.Postでその処理を投げるという仕組みになっている。

SyncronizationContextって何？という方はこちら
→ [【Unity開発者向け】「SynchronizationContext」と「Taskのawait」 \- Qiita](https://qiita.com/toRisouP/items/a2c1bb1b0c4f73366bc6)

なので、Progress.Reportを呼んだ段階ではSyncronizationContext内のキューに積まれるだけで、実際に実行されるタイミングは分からないという話。
Unityの実装がどうなってるかいまいちわからないけど、多分1フレームごとのライフサイクルのどっかでDequeueされる。
参考：[UnitySynchronizationContext.cs#L41-L61](https://github.com/Unity-Technologies/UnityCsReference/blob/54dd67f09e090b1ad5ba1f55886f327106406b0c/Runtime/Export/UnitySynchronizationContext.cs#L41-L61)
なので、1フレーム遅らせると解決したりする。

```cs
var req = UnityWebRequest.Get("http://example.com");
await req.SendWebRequest().ConfigureAwait(new Progress<float>(x => _text.text = x.ToString()));
await UniTask.DelayFrame(1);
Debug.Log("StartRequest Completed");
GameObject.Destroy(_text);
_text = null;
```

が、そもそもProgressでSynchronizationContextが使われているのが問題なので、**標準のProgressを使わなければ良い**。（ConfigureAwaitで求められるのはIProgress<float>なので、必ずしもProgress<float>でなくて良い）
で、SynchronizationContextを使わないProgressが既にUniRx.Asyncの中に組み込まれている。
→ [Progress.cs](https://github.com/neuecc/UniRx/blob/2b4202646aef80d905717614649353313c742f3e/Assets/Plugins/UniRx/Scripts/Async/Progress.cs)

```cs
var req = UnityWebRequest.Get("http://example.com");
await req.SendWebRequest().ConfigureAwait(Progress.Create<float>(x => _text.text = x.ToString()));
Debug.Log("StartRequest Completed");
GameObject.Destroy(_text);
_text = null;
```

これでNullReferenceExceptionは発生しなくなる。めでたしめでたし。

# まとめ

- UnityWebRequestは必要なら手動でAbortしよう
- 標準のProgressは使わず、UniRx.AsyncのProgress.Create<float>を使おう