GitLab

There is no reason to differentiate between sync/async
case. Additionally we may get rid of loading entirely, in which case
the async case would have to change.

TEST=covered by tests
BUG=468458

Review URL: https://codereview.chromium.org/1022583003

Cr-Commit-Position: refs/heads/master@{#321582}