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/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/test')
| -rw-r--r-- | sync/test/engine/fake_sync_scheduler.cc | 6 | ||||
| -rw-r--r-- | sync/test/engine/fake_sync_scheduler.h | 5 |
2 files changed, 3 insertions, 8 deletions
diff --git a/sync/test/engine/fake_sync_scheduler.cc b/sync/test/engine/fake_sync_scheduler.cc index 200edb0..585248e 100644 --- a/sync/test/engine/fake_sync_scheduler.cc +++ b/sync/test/engine/fake_sync_scheduler.cc @@ -6,16 +6,14 @@ namespace syncer { -FakeSyncScheduler::FakeSyncScheduler() - : created_on_loop_(base::MessageLoop::current()) {} +FakeSyncScheduler::FakeSyncScheduler() {} FakeSyncScheduler::~FakeSyncScheduler() {} void FakeSyncScheduler::Start(Mode mode) { } -void FakeSyncScheduler::RequestStop(const base::Closure& callback) { - created_on_loop_->PostTask(FROM_HERE, callback); +void FakeSyncScheduler::RequestStop() { } void FakeSyncScheduler::ScheduleLocalNudge( diff --git a/sync/test/engine/fake_sync_scheduler.h b/sync/test/engine/fake_sync_scheduler.h index 96805f5..f12e1df 100644 --- a/sync/test/engine/fake_sync_scheduler.h +++ b/sync/test/engine/fake_sync_scheduler.h @@ -20,7 +20,7 @@ class FakeSyncScheduler : public SyncScheduler { virtual ~FakeSyncScheduler(); virtual void Start(Mode mode) OVERRIDE; - virtual void RequestStop(const base::Closure& callback) OVERRIDE; + virtual void RequestStop() OVERRIDE; virtual void ScheduleLocalNudge( const base::TimeDelta& desired_delay, ModelTypeSet types, @@ -58,9 +58,6 @@ class FakeSyncScheduler : public SyncScheduler { virtual void OnShouldStopSyncingPermanently() OVERRIDE; virtual void OnSyncProtocolError( const sessions::SyncSessionSnapshot& snapshot) OVERRIDE; - - private: - base::MessageLoop* const created_on_loop_; }; } // namespace syncer |