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

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

Issue 2066093003: Refactor fetching profile avatar icons and labels repeated md-settings & md-user-manager (C++) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@avatar-selector-refactor
Patch Set: Addressed comments Created 4 years, 6 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_avatar_icon_util.h
diff --git a/chrome/browser/profiles/profile_avatar_icon_util.h b/chrome/browser/profiles/profile_avatar_icon_util.h
index 8d7ed521b60af556c9d5d4960f0c6c22e24269a4..5f1707a5930afe350a2625775f7f684c7f3e56a0 100644
--- a/chrome/browser/profiles/profile_avatar_icon_util.h
+++ b/chrome/browser/profiles/profile_avatar_icon_util.h
@@ -7,12 +7,14 @@
#include <stddef.h>
+#include <memory>
#include <string>
#include "third_party/skia/include/core/SkColor.h"
namespace base {
class FilePath;
+class ListValue;
}
namespace gfx {
@@ -107,6 +109,10 @@ bool IsDefaultAvatarIconUrl(const std::string& icon_url, size_t *icon_index);
bool GetImageURLWithThumbnailSize(
const GURL& old_url, int thumbnail_size, GURL* new_url);
+// Returns a list of dictionaries containing default profile avatar icons and
+// labels.
Mike Lerman 2016/06/20 00:42:54 Please note how this list is ordered. Also note -
Moe 2016/06/20 17:24:08 a. I amended the comment to note what the labels a
Mike Lerman 2016/06/20 17:38:02 Ah, those labels used to also be the default profi
+std::unique_ptr<base::ListValue> GetDefaultProfileAvatarIconsAndLabels();
+
} // namespace profiles
#endif // CHROME_BROWSER_PROFILES_PROFILE_AVATAR_ICON_UTIL_H_

Powered by Google App Engine
This is Rietveld 408576698