[Sync] Move StartAssociation from NonUIDataTypeController to SharedChangeProcessor.

components/sync/driver/shared_change_processor_unittest.cc

 // Copyright 2014 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 "components/sync/driver/shared_change_processor.h"
 
 #include <cstddef>
+#include <memory>
 #include <string>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/compiler_specific.h"
 #include "base/location.h"
 #include "base/single_thread_task_runner.h"
 #include "base/synchronization/waitable_event.h"
 #include "base/threading/thread.h"
 #include "components/sync/api/attachments/attachment_id.h"
@@ skipping 77 matching lines @@
 
   // FakeSyncClient override.
   base::WeakPtr<syncer::SyncableService> GetSyncableServiceForType(
       syncer::ModelType type) override {
     return db_syncable_service_->AsWeakPtr();
   }
 
  protected:
   void SetUp() override {
     test_user_share_.SetUp();
-    shared_change_processor_ = new SharedChangeProcessor();
+    shared_change_processor_ = new SharedChangeProcessor(syncer::AUTOFILL);
     ASSERT_TRUE(backend_thread_.Start());
     ASSERT_TRUE(backend_thread_.task_runner()->PostTask(
         FROM_HERE,
         base::Bind(&SyncSharedChangeProcessorTest::SetUpDBSyncableService,
                    base::Unretained(this))));
   }
 
   void TearDown() override {
     EXPECT_TRUE(backend_thread_.task_runner()->PostTask(
         FROM_HERE,
@@ skipping 66 matching lines @@
   }
 
   // Used by Connect().  The SharedChangeProcessor is passed in
   // because we modify |shared_change_processor_| on the main thread
   // (in TearDown()).
   void ConnectOnDBThread(
       const scoped_refptr<SharedChangeProcessor>& shared_change_processor) {
     DCHECK(backend_thread_.task_runner()->BelongsToCurrentThread());
     EXPECT_TRUE(shared_change_processor->Connect(
         this, &processor_factory_, test_user_share_.user_share(),
-        &error_handler_, syncer::AUTOFILL,
-        base::WeakPtr<syncer::SyncMergeResult>()));
+        &error_handler_, base::WeakPtr<syncer::SyncMergeResult>()));
     did_connect_ = true;
   }
 
   void CheckAttachmentServiceOnDBThread(base::WaitableEvent* event) {
     DCHECK(backend_thread_.task_runner()->BelongsToCurrentThread());
     DCHECK(db_syncable_service_.get());
     has_attachment_service_ = !!db_syncable_service_->attachment_service();
     event->Signal();
   }
 
@@ skipping 24 matching lines @@
 // creates AttachmentService and passes it back to the syncable service.
 TEST_F(SyncSharedChangeProcessorTest, ConnectWithAttachmentStore) {
   SetAttachmentStore();
   Connect();
   EXPECT_TRUE(HasAttachmentService());
 }
 
 }  // namespace
 
 }  // namespace sync_driver