Clean up LoginUIService. Remove Singleton behavior from PeopleHandler

components/browser_sync/browser/profile_sync_service.cc

Index: components/browser_sync/browser/profile_sync_service.cc
diff --git a/components/browser_sync/browser/profile_sync_service.cc b/components/browser_sync/browser/profile_sync_service.cc
index 0d2c7eb3ce6f602145fd073bf12f2e3efcc57330..5d0e9239003120384ab7d9acf0539e625d2f923e 100644
--- a/components/browser_sync/browser/profile_sync_service.cc
+++ b/components/browser_sync/browser/profile_sync_service.cc
@@ -1704,6 +1704,8 @@ void ProfileSyncService::ChangePreferredDataTypes(
 
   // Now reconfigure the DTM.
   ReconfigureDatatypeManager();
+
+  NotifyObservers();

[Nicolas Zea, 2016/05/17 20:00:55]

Is this necessary? Configuration should notify the observer anyways.

[tommycli, 2016/05/17 23:23:22]

Hi, I added this because OnUserChoseDatatypes wasn't notifying the observers
previously.

I looked through the code for the ReconfigureDatatypeManager and it also didn't
look like there was any path that notified the observers. (Unless there was an
unrecoverable error)

 }
 
 syncer::ModelTypeSet ProfileSyncService::GetActiveDataTypes() const {