GitLab

In preparation for new draw functor which does not have synchronous
invoke to free render thread resources, change all resource return to
posting to UI thread instead.

Correctness is relatively simple: if BrowserViewRenderer is destroyed
then there is no need to return any resources. Otherwise resources
are returned eventually. Detach/shutdown has the same logic. RTM just
neends to ensure all resource returns are posted.

There are some potential perf downsides:
There will be more tasks on the UI thread. This is not really avoidable.

Can no longer synchronous return resources before starting a new frame.
This may cause higher resource usage. This is technically avoidable
but is technically complicated, so go with the simple solution first.

Bug: 900965
Change-Id: Id34d05a575799f2cf22166da595ba5e062c0078c
Reviewed-on: https://chromium-review.googlesource.com/c/1385854Reviewed-by: Tobias Sargeant <tobiasjs@chromium.org>
Commit-Queue: Bo <boliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#619982}