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

tools/heapcheck/suppressions.txt

Index: tools/heapcheck/suppressions.txt
diff --git a/tools/heapcheck/suppressions.txt b/tools/heapcheck/suppressions.txt
index 7b2563c3718d77d6156c89d0312730d3911f3b46..ac35905d5f0cc3f6b5af63057189dde136df27b5 100644
--- a/tools/heapcheck/suppressions.txt
+++ b/tools/heapcheck/suppressions.txt
@@ -1814,40 +1814,6 @@
    fun:chromeos::UserManagerTest_*_Test::TestBody
 }
 {
-   bug_117098_a
-   Heapcheck:Leak
-   fun:base::internal::WeakReferenceOwner::GetRef
-   fun:base::SupportsWeakPtr::AsWeakPtr
-   fun:ProfileSyncServicePreferenceTest::StartSyncService
-}
-{
-   bug_117098_b
-   Heapcheck:Leak
-   fun:__gnu_cxx::new_allocator::allocate
-   fun:std::_Vector_base::_M_allocate
-   fun:std::vector::_M_insert_aux
-   fun:std::vector::push_back
-   fun:browser_sync::GenericChangeProcessor::ApplyChangesFromSyncModel
-}
-{
-   bug_117098_c
-   Heapcheck:Leak
-   fun:CreateAndSaveChangeProcessorActionP::gmock_Impl::gmock_PerformImpl
-   fun:testing::internal::ActionHelper::Perform
-   fun:CreateAndSaveChangeProcessorActionP::gmock_Impl::Perform
-   fun:testing::Action::Perform
-   fun:testing::internal::ActionResultHolder::PerformAction
-   fun:testing::internal::FunctionMockerBase::UntypedPerformAction
-   fun:testing::internal::UntypedFunctionMockerBase::UntypedInvokeWith
-   fun:testing::internal::FunctionMockerBase::InvokeWith
-   fun:testing::internal::FunctionMocker::Invoke
-   fun:ProfileSyncComponentsFactoryMock::CreateGenericChangeProcessor
-   fun:browser_sync::UIDataTypeController::Associate
-   fun:browser_sync::UIDataTypeController::Start
-   fun:browser_sync::DataTypeManagerImpl::StartNextType
-   fun:browser_sync::DataTypeManagerImpl::DownloadReady
-}
-{
    bug_117389
    Heapcheck:Leak
    ...