GitLab

This reverts commit d82af7a9.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 842133 as the
culprit for failures in the build cycles as shown on:
https://analysis.chromium.org/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtL2Q4MmFmN2E5ZWVmYzI1ZGRmMTI0ZmMxMjg0YTM3NmVkYzdmZGNmNWMM

Sample Failed Build: https://ci.chromium.org/b/8858364635543817904

Sample Failed Step: ash_unittests

Original change's description:
> ambient: construct photo_controller on enabled
> 
> AmbientPhotoController and AmbientPhotoCache are only necessary
> when a user is logged in and ambient mode is enabled. Construct them
> at that time, and destruct them if ambient mode is disabled.
> 
> Schedule fetching backup images when AmbientPhotoController is
> constructed, rather than in OnFirstSessionStarted.
> 
> BUG=b:176094707
> TEST=ash_unittests --gtest_filter="AmbientControllerTest.*"
> 
> Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome
> Change-Id: Id87bb16e6d4cc14074dd9bdb352fb5795d0f8915
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2605744
> Commit-Queue: Jeffrey Young <cowmoo@chromium.org>
> Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
> Reviewed-by: Tao Wu <wutao@chromium.org>
> Reviewed-by: Jeroen Dhollander <jeroendh@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#842133}


No-Presubmit: true
No-Tree-Checks: true
No-Try: true
BUG=b:176094707
Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome

Change-Id: Ieccf27239a7fb618deb72fe6ebe9519462df7add
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2623527
Cr-Commit-Position: refs/heads/master@{#842283}