diff options
author | haitaol@chromium.org <haitaol@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-08 02:18:31 +0000 |
---|---|---|
committer | haitaol@chromium.org <haitaol@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-08 02:18:31 +0000 |
commit | 8679c1174470e6feac4f90cb7ffa7ae8151d8e77 (patch) | |
tree | 9d9e4fb3dae1ec073cf22d13b3c6c0e3f1099a31 /sync/internal_api/test | |
parent | 3cb7cfeca270780ade12d147b45a256376ca55e5 (diff) | |
download | chromium_src-8679c1174470e6feac4f90cb7ffa7ae8151d8e77.zip chromium_src-8679c1174470e6feac4f90cb7ffa7ae8151d8e77.tar.gz chromium_src-8679c1174470e6feac4f90cb7ffa7ae8151d8e77.tar.bz2 |
Lock-free shutdown of profile sync service. Changes include:
* Disconnect non-frontend processor/associator to stop accessing directory
so that sync backend can be shut down without waiting.
* Change non-frontend controller so that creation/destruction of
processor/associator doesn't depend on valid controller. So
scoped wait on stopping controller can be removed.
* Move sync thread to PSS. It's created when starting first backend and
destroyed on last browser thread.
BUG=19757
Review URL: https://chromiumcodereview.appspot.com/16770005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@210333 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sync/internal_api/test')
-rw-r--r-- | sync/internal_api/test/fake_sync_manager.cc | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/sync/internal_api/test/fake_sync_manager.cc b/sync/internal_api/test/fake_sync_manager.cc index 7a6d6a4..24c228c 100644 --- a/sync/internal_api/test/fake_sync_manager.cc +++ b/sync/internal_api/test/fake_sync_manager.cc @@ -210,10 +210,7 @@ void FakeSyncManager::SaveChanges() { // Do nothing. } -void FakeSyncManager::StopSyncingForShutdown(const base::Closure& callback) { - if (!sync_task_runner_->PostTask(FROM_HERE, callback)) { - NOTREACHED(); - } +void FakeSyncManager::StopSyncingForShutdown() { } void FakeSyncManager::ShutdownOnSyncThread() { |