The AutofillProfileSyncableService's lifetime should be managed by the WebDataService

chrome/browser/sync/abstract_profile_sync_service_test.cc

 // 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.
 
 #include "chrome/browser/sync/abstract_profile_sync_service_test.h"
 
 #include "base/location.h"
 #include "chrome/browser/sync/internal_api/write_transaction.h"
 #include "chrome/browser/sync/protocol/sync.pb.h"
 #include "chrome/browser/sync/syncable/directory_manager.h"
@@ skipping 21 matching lines @@
 using syncable::UNITTEST;
 using syncable::WriteTransaction;
 
 /* static */
 const std::string ProfileSyncServiceTestHelper::GetTagForType(
     ModelType model_type) {
   return syncable::ModelTypeToRootTag(model_type);
 }
 
 /* static */
-bool ProfileSyncServiceTestHelper::CreateRoot(
-    ModelType model_type, UserShare* user_share,
-    TestIdFactory* ids) {
+bool ProfileSyncServiceTestHelper::CreateRoot(ModelType model_type,

[akalin, 2011/10/10 21:58:57]

revert these diffs please (as far as I can tell, they're just changing
indentation)

[Ilya Sherman, 2011/10/10 22:33:13]

Done.

+                                              UserShare* user_share,
+                                              TestIdFactory* ids) {
   DirectoryManager* dir_manager = user_share->dir_manager.get();
 
   ScopedDirLookup dir(dir_manager, user_share->name);
   if (!dir.good())
     return false;
 
   std::string tag_name = GetTagForType(model_type);
 
   WriteTransaction wtrans(FROM_HERE, UNITTEST, dir);
   MutableEntry node(&wtrans,
@@ skipping 57 matching lines @@
   ui_loop_.RunAllPending();
 }
 
 bool AbstractProfileSyncServiceTest::CreateRoot(ModelType model_type) {
   return ProfileSyncServiceTestHelper::CreateRoot(
       model_type,
       service_->GetUserShare(),
       service_->id_factory());
 }
 
-CreateRootTask::CreateRootTask(
-    AbstractProfileSyncServiceTest* test, ModelType model_type)
-    : test_(test), model_type_(model_type), success_(false) {
+CreateRootTask::CreateRootTask(AbstractProfileSyncServiceTest* test,
+                               ModelType model_type)
+    : test_(test),
+      model_type_(model_type),
+      success_(false) {
 }
 
-CreateRootTask::~CreateRootTask() {}
+CreateRootTask::~CreateRootTask() {
+}
+
 void CreateRootTask::Run() {
   success_ = test_->CreateRoot(model_type_);
 }
 
 bool CreateRootTask::success() {
   return success_;
 }