GitLab

This reverts commit 7f596585.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 568084 as the
culprit for failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzdmNTk2NTg1YTM4MmI3ZDIzNDQ3NDAxOTY4OTU4OGUxOTE5ZDU2OWYM

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.memory/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/27973

Sample Failed Step: unit_tests

Original change's description:
> cros: Make LoginDisplayHost own LoginDisplay
> 
> Instead of this ownership model
> 
>   LoginDisplayHost owns ExistingUserController owns LoginDisplay
>   LoginDisplayHost owns WebUIScreenLocker owns LoginDisplay
> 
> switch to
> 
>   LoginDisplayHost owns ExistingUserController, LoginDisplay
> 
> This makes it clear that the LoginDisplay instance has one owner, and
> that it shares the same lifetime as LoginDisplayHost.
> 
> Bug: 851012
> Change-Id: I80cb56d8fc244ce5331046ba31defba4cd77c45f
> Reviewed-on: https://chromium-review.googlesource.com/1093741
> Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
> Commit-Queue: Jacob Dufault <jdufault@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#568084}

Change-Id: I144f702fc39267db2ef830b276ca3c77c2c3c262
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 851012
Reviewed-on: https://chromium-review.googlesource.com/1105240
Cr-Commit-Position: refs/heads/master@{#568182}