sync: Consistently refcount ModelSafeWorkers

chrome/browser/sync/glue/sync_backend_host_core.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_SYNC_GLUE_SYNC_BACKEND_HOST_CORE_H_
 #define CHROME_BROWSER_SYNC_GLUE_SYNC_BACKEND_HOST_CORE_H_
 
 #include "base/memory/ref_counted.h"
 
 #include "base/timer/timer.h"
 #include "chrome/browser/sync/glue/chrome_encryptor.h"
 #include "chrome/browser/sync/glue/sync_backend_host_impl.h"
 #include "sync/internal_api/public/base/cancelation_signal.h"
 #include "sync/internal_api/public/sync_encryption_handler.h"
 #include "url/gurl.h"
 
 namespace browser_sync {
 
 class SyncBackendHostImpl;
 
 // Utility struct for holding initialization options.
 struct DoInitializeOptions {
   DoInitializeOptions(
       base::MessageLoop* sync_loop,
       SyncBackendRegistrar* registrar,
       const syncer::ModelSafeRoutingInfo& routing_info,
-      const std::vector<syncer::ModelSafeWorker*>& workers,
+      const std::vector<scoped_refptr<syncer::ModelSafeWorker> >& workers,
       const scoped_refptr<syncer::ExtensionsActivity>& extensions_activity,
       const syncer::WeakHandle<syncer::JsEventHandler>& event_handler,
       const GURL& service_url,
       scoped_ptr<syncer::HttpPostProviderFactory> http_bridge_factory,
       const syncer::SyncCredentials& credentials,
       const std::string& invalidator_client_id,
       scoped_ptr<syncer::SyncManagerFactory> sync_manager_factory,
       bool delete_sync_data_folder,
       const std::string& restored_key_for_bootstrapping,
       const std::string& restored_keystore_key_for_bootstrapping,
       scoped_ptr<syncer::InternalComponentsFactory>
           internal_components_factory,
       scoped_ptr<syncer::UnrecoverableErrorHandler>
           unrecoverable_error_handler,
       syncer::ReportUnrecoverableErrorFunction
           report_unrecoverable_error_function);
   ~DoInitializeOptions();
 
   base::MessageLoop* sync_loop;
   SyncBackendRegistrar* registrar;
   syncer::ModelSafeRoutingInfo routing_info;
-  std::vector<syncer::ModelSafeWorker*> workers;
+  std::vector<scoped_refptr<syncer::ModelSafeWorker> > workers;
   scoped_refptr<syncer::ExtensionsActivity> extensions_activity;
   syncer::WeakHandle<syncer::JsEventHandler> event_handler;
   GURL service_url;
   // Overridden by tests.
   scoped_ptr<syncer::HttpPostProviderFactory> http_bridge_factory;
   syncer::SyncCredentials credentials;
   const std::string invalidator_client_id;
   scoped_ptr<syncer::SyncManagerFactory> sync_manager_factory;
   std::string lsid;
   bool delete_sync_data_folder;
@@ skipping 226 matching lines @@
   const bool has_sync_setup_completed_;
 
   base::WeakPtrFactory<SyncBackendHostCore> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(SyncBackendHostCore);
 };
 
 }  // namespace browser_sync
 
 #endif  // CHROME_BROWSER_SYNC_GLUE_SYNC_BACKEND_HOST_CORE_H_