Move QuotaManager from DB thread to task scheduler
QuotaManager already takes a SequencedTaskRunner as a parameter, so only
need to update constructor call sites. Most call sites are from unit
tests through MockQuotaManager.
Then need to fix a lot of unit tests that explicitly controls RunLoop.
storage_unittests are generally from MessageLoop to
ScopedTaskEnvironment. Upper layer tests are switched to
TestBrowserThreadBundle, and RunLoop switched to
RunAllBlockingPoolTasksUntilIdle.
Note also needed to add some RunUntilIdle calls near end of some tests
to make sure sql db is shutdown on task scheduler before underlying file
is deleted.
Bug: 689520
Change-Id: Ie186475ca09250b25a4020fcc0c4c216efa2d7a5
Reviewed-on: https://chromium-review.googlesource.com/583852
Reviewed-by: Michael Nordman <michaeln@chromium.org>
Reviewed-by: Joshua Bell <jsbell@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Bo Liu <boliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#489777}26 files changed
