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

Unified Diff: chrome/browser/profiles/profile_info_cache.h

Issue 212603011: Newly created profiles should have the grey silhouette as avatar. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove duplicate references to resource for grey avatar. Created 6 years, 9 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/profiles/profile_info_cache.h
diff --git a/chrome/browser/profiles/profile_info_cache.h b/chrome/browser/profiles/profile_info_cache.h
index 28251f08605adeb5c0f303e33f396c85cbcf00e9..49fbcfecc2b562407d43d4b3fa3684029694ff03 100644
--- a/chrome/browser/profiles/profile_info_cache.h
+++ b/chrome/browser/profiles/profile_info_cache.h
@@ -128,8 +128,12 @@ class ProfileInfoCache : public ProfileInfoInterface,
// Gets the number of default avatar icons that exist.
static size_t GetDefaultAvatarIconCount();
+ // Gets the index for the (grey silhouette) avatar used as a placeholder.
+ static int GetPlaceholderAvatarIndex();
// Gets the resource ID of the default avatar icon at |index|.
static int GetDefaultAvatarIconResourceIDAtIndex(size_t index);
+ // Gets the resource ID of the placeholder avatar icon.
+ static int GetPlaceholderAvatarIconResourceID();
noms (inactive) 2014/03/27 20:19:32 Why do you need both methods? Isn't GetPlaceholde
Mike Lerman 2014/04/02 17:46:25 Yes, but this removes the need to have those two n
// Returns a URL for the default avatar icon with specified index.
static std::string GetDefaultAvatarIconUrl(size_t index);
// Checks if |index| is a valid avatar icon index

Powered by Google App Engine
This is Rietveld 408576698