forked from chromium/chromium
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
personalization: Take no action on failed Google Photos wallpaper req…
…uest Currently, if a request fails when trying to fetch the URL for a Google Photos wallpaper, we reset to the default wallpaper. We only want to reset the wallpaper if the photo does not exist. We don't want to reset the wallpaper if the request failed entirely, so we need a way to differentiate between those two error states in the controller. Otherwise, the user disconnecting from WiFi could cause the wallpaper to reset unnecessarily. This would be a rare occurrence right now, but once an upcoming CL enables periodic staleness checking for Google Photos wallpapers, it would happen any time a user is disconnected from WiFi for a substantial period of time. Bug: b:226198384 Change-Id: I1e60971a1ace376cf6b26a421fda2f0cdf072d39 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3543355 Reviewed-by: Colin Kincaid <ckincaid@chromium.org> Reviewed-by: David Black <dmblack@google.com> Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org> Commit-Queue: Angus McLean <angusmclean@chromium.org> Cr-Commit-Position: refs/heads/main@{#985654}
- Loading branch information
Angus L. M. McLean IV
authored and
Chromium LUCI CQ
committed
Mar 26, 2022
1 parent
9040929
commit 7e0a193
Showing
9 changed files
with
66 additions
and
19 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters