[Sync] Have SyncableService's take ownership of their SyncChangeProcessor.

chrome/browser/extensions/app_notification_manager.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_APP_NOTIFICATION_MANAGER_H_
 #define CHROME_BROWSER_EXTENSIONS_APP_NOTIFICATION_MANAGER_H_
 #pragma once
 
 #include <map>
 
@@ skipping 55 matching lines @@
   virtual SyncDataList GetAllSyncData(syncable::ModelType type) const OVERRIDE;
   // Process notifications related changes from Sync, merging them into
   // our model.
   virtual SyncError ProcessSyncChanges(
       const tracked_objects::Location& from_here,
       const SyncChangeList& change_list) OVERRIDE;
   // Associate and merge sync data model with our data model.
   virtual SyncError MergeDataAndStartSyncing(
       syncable::ModelType type,
       const SyncDataList& initial_sync_data,
-      SyncChangeProcessor* sync_processor) OVERRIDE;
+      scoped_ptr<SyncChangeProcessor> sync_processor) OVERRIDE;
   virtual void StopSyncing(syncable::ModelType type) OVERRIDE;
 
  private:
   friend class AppNotificationManagerSyncTest;
   FRIEND_TEST_ALL_PREFIXES(AppNotificationManagerSyncTest,
                            NotificationToSyncDataToNotification);
   FRIEND_TEST_ALL_PREFIXES(AppNotificationManagerSyncTest,
                            GetAllSyncDataNoLocal);
   FRIEND_TEST_ALL_PREFIXES(AppNotificationManagerSyncTest,
                            GetAllSyncDataSomeLocal);
@@ skipping 61 matching lines @@
       const SyncData& sync_data);
 
   Profile* profile_;
   content::NotificationRegistrar registrar_;
   scoped_ptr<NotificationMap> notifications_;
 
   // This should only be used on the FILE thread.
   scoped_ptr<AppNotificationStorage> storage_;
 
   // Sync change processor we use to push all our changes.
-  SyncChangeProcessor* sync_processor_;
+  scoped_ptr<SyncChangeProcessor> sync_processor_;
   // Whether the sync model is associated with the local model.
   // In other words, whether we are ready to apply sync changes.
   bool models_associated_;
   // Whether syncer changes are being processed right now.
   bool processing_syncer_changes_;
 
   // Used for a histogram of load time.
   scoped_ptr<PerfTimer> load_timer_;
 
   DISALLOW_COPY_AND_ASSIGN(AppNotificationManager);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_APP_NOTIFICATION_MANAGER_H_