GitLab

This reverts commit d42167eb.

Reason for revert: This seems to have made ProfilePickerEnterpriseCreationFlowBrowserTest.CreateSignedInProfileSettings fail on win7 bot: https://ci.chromium.org/p/chromium/builders/ci/Win7%20%2832%29%20Tests/65702 . Output: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8865640044637153696/+/steps/browser_tests/0/logs/Deterministic_failure:_ProfilePickerEnterpriseCreationFlowBrowserTest.CreateSignedInProfileSettings__status_CRASH_/0

Original change's description:
> [Dice sync delegate] Split sync confirmation and sync disabled
>
> This CL splits the ShowSyncConfirmation() function of the dice turn on
> sync helper delegate into one that shows the normal sync confirmation
> and one that shows a error message that sync is disabled by the admin.
>
> Since both the dialogs are implemented by the same webUI page, there
> is not much behavioral change. The only change is for the delegate
> for the signed-in profile creation flow which opens a new browser
> window and shows the dialog in this window instead of showing it in
> the creation flow window.
>
> In follow-up CLs, the variant of the UI will be passed as a param
> to the WebUI page replacing the logic to decide which variant to show.
>
> Bug: 1126913
> Change-Id: Ibac9a339c4fb071f960f15864482c818247f5719
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2487100
> Commit-Queue: Jan Krcal <jkrcal@chromium.org>
> Reviewed-by: David Roger <droger@chromium.org>
> Auto-Submit: Jan Krcal <jkrcal@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#820222}

TBR=droger@chromium.org,jkrcal@chromium.org

Change-Id: I81c91215340e0d77530c3e1faf0a23eac31722b9
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1126913
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2495329Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#820268}