[Autofill] Run autofill-profile de-dupe after sync starts if sync is enabled.

components/autofill/core/browser/autofill_manager.cc

Index: components/autofill/core/browser/autofill_manager.cc
diff --git a/components/autofill/core/browser/autofill_manager.cc b/components/autofill/core/browser/autofill_manager.cc
index e43e85b26d8268c3f1c56f4bc09cb35c5e81517b..9e7330b7c8132f0b2a9611da804b34ced9b8f212 100644
--- a/components/autofill/core/browser/autofill_manager.cc
+++ b/components/autofill/core/browser/autofill_manager.cc
@@ -215,6 +215,8 @@ AutofillManager::AutofillManager(
     download_manager_.reset(new AutofillDownloadManager(driver, this));
   }
   CountryNames::SetLocaleString(app_locale_);
+  if (personal_data_ && client_)
+    personal_data_->OnSyncServiceConfigured(client_->GetSyncService());

[Nicolas Zea, 2016/07/12 20:04:47]

How do you know that the sync service is configured at this point in time? Kind
of seems like a strange place to make this call, but I'm not too familiar with
the autofill initialization process.

[Roger McFarlane (Chromium), 2016/07/12 21:23:26]

Because we're in the context of the AutofillManager, which is given a bunch of
existing and otherwise usable things to manage.

OnSyncServiceConfigured might be an overly strong name. Perhaps it would be
better named "OnValidSyncServicePointerAvailable()"

:)

All I really need at this point is a valid SyncService instance that I can query
about whether it will eventually be doing autofill profile syncing.

 }
 
 AutofillManager::~AutofillManager() {}