sync: Consistently refcount ModelSafeWorkers

sync/internal_api/test/test_internal_components_factory.cc

 // Copyright (c) 2012 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.
 
 #include "sync/internal_api/public/test/test_internal_components_factory.h"
 
 #include "sync/sessions/sync_session_context.h"
 #include "sync/syncable/in_memory_directory_backing_store.h"
 #include "sync/syncable/on_disk_directory_backing_store.h"
 #include "sync/syncable/invalid_directory_backing_store.h"
@@ skipping 14 matching lines @@
     const std::string& name,
     sessions::SyncSessionContext* context,
     syncer::CancelationSignal* cancelation_signal) {
   return scoped_ptr<SyncScheduler>(new FakeSyncScheduler());
 }
 
 scoped_ptr<sessions::SyncSessionContext>
 TestInternalComponentsFactory::BuildContext(
     ServerConnectionManager* connection_manager,
     syncable::Directory* directory,
-    const std::vector<ModelSafeWorker*>& workers,
+    const std::vector<scoped_refptr<ModelSafeWorker> >& workers,
     ExtensionsActivity* monitor,
     const std::vector<SyncEngineEventListener*>& listeners,
     sessions::DebugInfoGetter* debug_info_getter,
     TrafficRecorder* traffic_recorder,
     const std::string& invalidator_client_id) {
 
   // Tests don't wire up listeners.
   std::vector<SyncEngineEventListener*> empty_listeners;
   return scoped_ptr<sessions::SyncSessionContext>(
       new sessions::SyncSessionContext(
@@ skipping 25 matching lines @@
   NOTREACHED();
   return scoped_ptr<syncable::DirectoryBackingStore>();
 }
 
 InternalComponentsFactory::Switches
 TestInternalComponentsFactory::GetSwitches() const {
   return switches_;
 }
 
 }  // namespace syncer