Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7265)

Unified Diff: chrome/browser/resources/chromeos/wallpaper_manager/js/event_page.js

Issue 162393002: Enable surprise me wallpaper for new profiles. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/chromeos/wallpaper_manager/js/event_page.js
diff --git a/chrome/browser/resources/chromeos/wallpaper_manager/js/event_page.js b/chrome/browser/resources/chromeos/wallpaper_manager/js/event_page.js
index 71062033f168ba4bc7d028f2f65f0f07528ae82b..52b9e62a416dcc7bf7e2b846f3136306b635fde9 100644
--- a/chrome/browser/resources/chromeos/wallpaper_manager/js/event_page.js
+++ b/chrome/browser/resources/chromeos/wallpaper_manager/js/event_page.js
@@ -274,3 +274,16 @@ chrome.storage.onChanged.addListener(function(changes, namespace) {
chrome.alarms.onAlarm.addListener(function() {
SurpriseWallpaper.getInstance().next();
});
+
+/**
+ * Enables surprise me wallpaper iff it has not already been configured.
+ */
+chrome.wallpaperPrivate.onRequestEnableSurpriseMe.addListener(function() {
+ Constants.WallpaperSyncStorage.get(Constants.AccessSurpriseMeEnabledKey,
+ function(items) {
+ if (!items.hasOwnProperty(Constants.AccessSurpriseMeEnabledKey)) {
Nikita (slow) 2014/04/08 16:58:27 I wonder if that's the right time to check for thi
Harry McCleave 2014/04/09 02:41:33 My impression was that if this sync fails for what
bshe 2014/04/09 14:32:15 Is this going to be called when an existing user l
+ WallpaperUtil.saveToStorage(Constants.AccessSurpriseMeEnabledKey,
+ true, true);
+ }
+ });
+});

Powered by Google App Engine
This is Rietveld 408576698