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

Unified Diff: chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.h

Issue 1497973002: This CL replaces e-mail with AccountId in wallpaper manager code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update after review. Created 5 years 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/chromeos/login/users/wallpaper/wallpaper_manager.h
diff --git a/chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.h b/chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.h
index 5aefa6a938524b26a11ed1e5cff33fa30fdafdb1..63efdb76bc208951f32da14a758578bfcdb6ec62 100644
--- a/chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.h
+++ b/chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.h
@@ -70,19 +70,19 @@ class WallpaperManager :
const content::NotificationSource& source,
const content::NotificationDetails& details) override;
- // Removes all |user_id| related wallpaper info and saved wallpapers.
- void RemoveUserWallpaperInfo(const std::string& user_id) override;
+ // Removes all |account_id| related wallpaper info and saved wallpapers.
+ void RemoveUserWallpaperInfo(const AccountId& account_id) override;
// Called when the policy-set wallpaper has been fetched. Initiates decoding
// of the JPEG |data| with a callback to SetPolicyControlledWallpaper().
void OnPolicyFetched(const std::string& policy,
- const std::string& user_id,
+ const AccountId& account_id,
scoped_ptr<std::string> data) override;
// Saves custom wallpaper to file, post task to generate thumbnail and updates
// local state preferences. If |update_wallpaper| is false, don't change
// wallpaper but only update cache.
- void SetCustomWallpaper(const std::string& user_id,
+ void SetCustomWallpaper(const AccountId& account_id,
const std::string& user_id_hash,
const std::string& file,
wallpaper::WallpaperLayout layout,
@@ -91,29 +91,29 @@ class WallpaperManager :
bool update_wallpaper) override;
// Sets wallpaper to default wallpaper (asynchronously with zero delay).
- void SetDefaultWallpaperNow(const std::string& user_id) override;
+ void SetDefaultWallpaperNow(const AccountId& account_id) override;
// Sets wallpaper to default wallpaper (asynchronously with default delay).
- void SetDefaultWallpaperDelayed(const std::string& user_id) override;
+ void SetDefaultWallpaperDelayed(const AccountId& account_id) override;
// Sets wallpaper to default.
void DoSetDefaultWallpaper(
- const std::string& user_id,
+ const AccountId& account_id,
wallpaper::MovableOnDestroyCallbackHolder on_finish) override;
- // Sets selected wallpaper information for |user_id| and saves it to Local
+ // Sets selected wallpaper information for |account_id| and saves it to Local
// State if |is_persistent| is true.
- void SetUserWallpaperInfo(const std::string& user_id,
+ void SetUserWallpaperInfo(const AccountId& account_id,
const wallpaper::WallpaperInfo& info,
bool is_persistent) override;
// Creates new PendingWallpaper request (or updates currently pending).
- void ScheduleSetUserWallpaper(const std::string& user_id,
+ void ScheduleSetUserWallpaper(const AccountId& account_id,
bool delayed) override;
// Sets wallpaper to |image| (asynchronously with zero delay). If
// |update_wallpaper| is false, skip change wallpaper but only update cache.
- void SetWallpaperFromImageSkia(const std::string& user_id,
+ void SetWallpaperFromImageSkia(const AccountId& account_id,
const gfx::ImageSkia& image,
wallpaper::WallpaperLayout layout,
bool update_wallpaper) override;
@@ -134,7 +134,7 @@ class WallpaperManager :
// Returns modifiable PendingWallpaper.
// Returns pending_inactive_ or creates new PendingWallpaper if necessary.
- PendingWallpaper* GetPendingWallpaper(const std::string& user_id,
+ PendingWallpaper* GetPendingWallpaper(const AccountId& account_id,
bool delayed);
// This is called by PendingWallpaper when load is finished.
@@ -142,14 +142,14 @@ class WallpaperManager :
// WallpaperManagerBase overrides:
void InitializeRegisteredDeviceWallpaper() override;
- bool GetUserWallpaperInfo(const std::string& user_id,
+ bool GetUserWallpaperInfo(const AccountId& account_id,
wallpaper::WallpaperInfo* info) const override;
- void OnWallpaperDecoded(const std::string& user_id,
+ void OnWallpaperDecoded(const AccountId& account_id,
wallpaper::WallpaperLayout layout,
bool update_wallpaper,
wallpaper::MovableOnDestroyCallbackHolder on_finish,
const user_manager::UserImage& user_image) override;
- void StartLoad(const std::string& user_id,
+ void StartLoad(const AccountId& account_id,
const wallpaper::WallpaperInfo& info,
bool update_wallpaper,
const base::FilePath& wallpaper_path,

Powered by Google App Engine
This is Rietveld 408576698