diff options
| author | rlarocque@chromium.org <rlarocque@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-11 03:11:06 +0000 |
|---|---|---|
| committer | rlarocque@chromium.org <rlarocque@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-11 03:11:06 +0000 |
| commit | f8197029e47adb3512609ec18f5aa7f3059befd3 (patch) | |
| tree | 55aca64c9cf27ff7a9dc0cbc3a401ae27692dfba /sync/internal_api/sync_manager_impl.h | |
| parent | 2db1e9f2c1502d384c9e5f849cfc85eec7875787 (diff) | |
| download | chromium_src-f8197029e47adb3512609ec18f5aa7f3059befd3.zip chromium_src-f8197029e47adb3512609ec18f5aa7f3059befd3.tar.gz chromium_src-f8197029e47adb3512609ec18f5aa7f3059befd3.tar.bz2 | |
sync: Consistently refcount ModelSafeWorkers
Certain interfaces in sync had signatures that included vectors of raw
pointers to ModelSafeWorkers. The use of raw pointers to refcounted
objects looks a lot like a bug waiting to happen.
This CL replaces any use of std::vector<ModelSafeWorker*> with
std::vector<scoped_refptr<ModelSafeWorker> >. This is not expected to
alter sync behavior in any significant way.
BUG=332251
Review URL: https://codereview.chromium.org/130193002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@244329 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sync/internal_api/sync_manager_impl.h')
| -rw-r--r-- | sync/internal_api/sync_manager_impl.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sync/internal_api/sync_manager_impl.h b/sync/internal_api/sync_manager_impl.h index 5d5cfaa..ed758bc 100644 --- a/sync/internal_api/sync_manager_impl.h +++ b/sync/internal_api/sync_manager_impl.h @@ -69,7 +69,7 @@ class SYNC_EXPORT_PRIVATE SyncManagerImpl : int sync_server_port, bool use_ssl, scoped_ptr<HttpPostProviderFactory> post_factory, - const std::vector<ModelSafeWorker*>& workers, + const std::vector<scoped_refptr<ModelSafeWorker> >& workers, ExtensionsActivity* extensions_activity, SyncManager::ChangeDelegate* change_delegate, const SyncCredentials& credentials, |