GitLab

There was a case when |UpdateSettings| could be called the first time
a user opened Ambient mode settings, even when Ambient mode was
disabled. Now, only update settings if pref is enabled.

Additionally, issue an |UpdateSettings| call when the pref is
enabled. This ensures that Ambient settings are synced to server even
when a user does not edit temperature settings or photo albums.

BUG=b:177456397
TEST=unit_tests --gtest_filter=AmbientModeHandlerTest.*
TEST=open settings page, personalization, screen saver, toggle ambient

Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome
Change-Id: I924af998954927f0fc38746b34b8fce1ff641509
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631305Reviewed-by: Jimmy Gong <jimmyxgong@chromium.org>
Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
Reviewed-by: Tao Wu <wutao@chromium.org>
Commit-Queue: Jeffrey Young <cowmoo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845050}