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

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

Issue 23480087: Use username_hash instead of email (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase 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.h
diff --git a/chrome/browser/chromeos/extensions/wallpaper_api.h b/chrome/browser/chromeos/extensions/wallpaper_api.h
index 6767d3f31a0e14a426781915ddb8f4969a5046d6..207436f6ceef8955646a161a39da614cb633d608 100644
--- a/chrome/browser/chromeos/extensions/wallpaper_api.h
+++ b/chrome/browser/chromeos/extensions/wallpaper_api.h
@@ -51,10 +51,11 @@ class WallpaperSetWallpaperFunction : public WallpaperFunctionBase {
std::string file_name_;
// Email address of logged in user.
- // TODO(bshe): User's email should not be used as part of wallpaper file path.
- // http://crbug.com/287020
std::string email_;
+ // Username hash of the logged in user.
+ std::string username_hash_;
Nikita (slow) 2013/09/25 14:47:11 nit: We try to migrate from email to user_id. Plea
bshe 2013/09/26 16:49:57 Done.
+
// String representation of downloaded wallpaper.
std::string image_data_;

Powered by Google App Engine
This is Rietveld 408576698