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

Unified Diff: chrome/browser/chromeos/extensions/wallpaper_api.cc

Issue 23480087: Use username_hash instead of email (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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/chromeos/extensions/wallpaper_api.cc
diff --git a/chrome/browser/chromeos/extensions/wallpaper_api.cc b/chrome/browser/chromeos/extensions/wallpaper_api.cc
index 1f0f785f93ce955ba61e1e17af318253be669d5a..f7b1a174de8d46cf5c98555989332f3cf07c654b 100644
--- a/chrome/browser/chromeos/extensions/wallpaper_api.cc
+++ b/chrome/browser/chromeos/extensions/wallpaper_api.cc
@@ -38,8 +38,10 @@ bool WallpaperSetWallpaperFunction::RunImpl() {
EXTENSION_FUNCTION_VALIDATE(!file_name_.empty());
- // Gets email address while at UI thread.
+ // Gets email address and username hash while at UI thread.
email_ = chromeos::UserManager::Get()->GetLoggedInUser()->email();
+ user_id_hash_ =
+ chromeos::UserManager::Get()->GetLoggedInUser()->username_hash();
image_data_.assign(input->GetBuffer(), input->GetSize());
StartDecode(image_data_);
@@ -55,7 +57,7 @@ void WallpaperSetWallpaperFunction::OnWallpaperDecoded(
image_data_.end());
chromeos::UserImage image(wallpaper, raw_image);
base::FilePath thumbnail_path = wallpaper_manager->GetCustomWallpaperPath(
- chromeos::kThumbnailWallpaperSubDir, email_, file_name_);
+ chromeos::kThumbnailWallpaperSubDir, user_id_hash_, file_name_);
sequence_token_ = BrowserThread::GetBlockingPool()->
GetNamedSequenceToken(chromeos::kWallpaperSequenceTokenName);
@@ -64,9 +66,10 @@ void WallpaperSetWallpaperFunction::OnWallpaperDecoded(
GetSequencedTaskRunnerWithShutdownBehavior(sequence_token_,
base::SequencedWorkerPool::BLOCK_SHUTDOWN);
- // In the new wallpaper picker UI, we do not depend on WallpaperDelegate
- // to refresh thumbnail. Uses a null delegate here.
- wallpaper_manager->SetCustomWallpaper(email_, file_name_, layout_,
+ wallpaper_manager->SetCustomWallpaper(email_,
+ user_id_hash_,
+ file_name_,
+ layout_,
chromeos::User::CUSTOMIZED,
image);
unsafe_wallpaper_decoder_ = NULL;
« no previous file with comments | « chrome/browser/chromeos/extensions/wallpaper_api.h ('k') | chrome/browser/chromeos/extensions/wallpaper_private_api.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698