The AutofillProfileSyncableService's lifetime should be managed by the WebDataService

chrome/browser/webdata/autofill_profile_syncable_service.h

 // Copyright (c) 2011 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_WEBDATA_AUTOFILL_PROFILE_SYNCABLE_SERVICE_H_
 #define CHROME_BROWSER_WEBDATA_AUTOFILL_PROFILE_SYNCABLE_SERVICE_H_
 #pragma once
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 12 matching lines @@
 #include "content/common/content_notification_types.h"
 #include "content/common/notification_observer.h"
 #include "content/common/notification_registrar.h"
 
 class AutofillProfile;
 class AutofillTable;
 class ProfileSyncServiceAutofillTest;
 class WebDataService;
 
 namespace browser_sync {
+class UnrecoverableErrorHandler;
+}
 
 extern const char kAutofillProfileTag[];
 
-class UnrecoverableErrorHandler;
-
 // The sync implementation for AutofillProfiles.
 // MergeDataAndStartSyncing() called first, it does cloud->local and
 // local->cloud syncs. Then for each cloud change we receive
 // ProcessSyncChanges() and for each local change Observe() is called.
 class AutofillProfileSyncableService
     : public SyncableService,
       public NotificationObserver,
       public base::NonThreadSafe {
  public:
   explicit AutofillProfileSyncableService(WebDataService* web_data_service);
@@ skipping 28 matching lines @@
   // Made virtual for ease of mocking in the unit-test.
   // Caller owns returned |profiles|.
   virtual bool LoadAutofillData(std::vector<AutofillProfile*>* profiles);
 
   // Helper to persist any changes that occured during model association to
   // the WebDatabase.
   // Made virtual for ease of mocking in the unit-test.
   virtual bool SaveChangesToWebData(const DataBundle& bundle);
 
  private:
-  friend class ::ProfileSyncServiceAutofillTest;
+  friend class ProfileSyncServiceAutofillTest;
   friend class MockAutofillProfileSyncableService;
   FRIEND_TEST_ALL_PREFIXES(AutofillProfileSyncableServiceTest,
                            MergeDataAndStartSyncing);
   FRIEND_TEST_ALL_PREFIXES(AutofillProfileSyncableServiceTest, GetAllSyncData);
   FRIEND_TEST_ALL_PREFIXES(AutofillProfileSyncableServiceTest,
                            ProcessSyncChanges);
   FRIEND_TEST_ALL_PREFIXES(AutofillProfileSyncableServiceTest,
                            ActOnChange);
 
   // The map of the guid to profiles owned by the |profiles_| vector.
@@ skipping 52 matching lines @@
 // This object is used in unit-tests as well, so it defined here.
 struct AutofillProfileSyncableService::DataBundle {
   DataBundle();
   ~DataBundle();
 
   std::vector<std::string> profiles_to_delete;
   std::vector<AutofillProfile*> profiles_to_update;
   std::vector<AutofillProfile*> profiles_to_add;
 };
 
-}  // namespace browser_sync
-
 #endif  // CHROME_BROWSER_WEBDATA_AUTOFILL_PROFILE_SYNCABLE_SERVICE_H_