Remove SyncService::OnUserChoseDatatypes

It has been replaced by SyncUserSettings::SetChosenDataTypes.
Note that *Profile*SyncService::OnUserChoseDatatypes still exists for
now (it's used by SyncUserSettings). This will be changed in a later CL.

This required a minor change in unified_consent_service_unittest.cc:
This test's implementation of TestSyncService used to override
OnUserChoseDatatypes. This logic is now moved (in slightly generalized
form) into TestSyncUserSettings.

Bug: 884159
Change-Id: I6723543f78a3f8c42f9bc29a9da84376ad92126b
Reviewed-on: https://chromium-review.googlesource.com/c/1353878
Reviewed-by: Thomas Tangl <tangltom@chromium.org>
Reviewed-by: Mohamed Amir Yosef <mamir@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#611721}
8 files changed