Reland "Rename ProfileSyncService to SyncServiceImpl"
Reason for reland: Original CL was reverted to unwind other changes.
Original change's description:
> Rename ProfileSyncService to SyncServiceImpl
>
> Renaming related classes like ProfileSyncServiceAndroid and
> ProfileSyncServiceHarness is left for next CLs. We also use this CL as
> an opportunity to hide the implementation class when possible:
> - Some includes for sync_service_impl.h are removed if unnecessary, or
> replaced with sync_service.h.
> - Even comments referring to SyncServiceImpl are updated to refer to
> the interface if the layer isn't aware SyncServiceImpl is being used.
>
> The CL also adds one missing include for <string> in
> fake_server_helper_android.cc to address one IWYU presubmit warning.
Bug: 1201272
Change-Id: I5355db7d07d1f8971ea942a273cbb94898dc72e9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2940282
Reviewed-by: David Roger <[email protected]>
Reviewed-by: Colin Blundell <[email protected]>
Commit-Queue: Victor Vianna <[email protected]>
Cr-Commit-Position: refs/heads/master@{#889192}
diff --git a/components/sync/driver/configure_context.h b/components/sync/driver/configure_context.h
index ffd9315..e87fb2b4 100644
--- a/components/sync/driver/configure_context.h
+++ b/components/sync/driver/configure_context.h
@@ -16,7 +16,7 @@
// Struct describing in which context sync was enabled, including state that can
// be assumed to be fixed while sync is enabled (or, more precisely, is
// representative of the last (re)configuration request). It's built by
-// ProfileSyncService and plumbed through DataTypeManager until datatype
+// SyncServiceImpl and plumbed through DataTypeManager until datatype
// controllers, which for USS datatypes propagate analogous information to the
// processor/bridge via DataTypeActivationRequest.
struct ConfigureContext {
