Refactor ProfileInfoCache in c/b/ui/views

chrome/browser/profiles/avatar_menu.cc

Index: chrome/browser/profiles/avatar_menu.cc
diff --git a/chrome/browser/profiles/avatar_menu.cc b/chrome/browser/profiles/avatar_menu.cc
index 64978fc40fb2159cfda36ee3c7b47367cfc62a3d..398cc31b85fa38af0d5ab92eb26ed6bc1e5e732a 100644
--- a/chrome/browser/profiles/avatar_menu.cc
+++ b/chrome/browser/profiles/avatar_menu.cc
@@ -40,15 +40,17 @@
 
 using content::BrowserThread;
 
-AvatarMenu::AvatarMenu(ProfileInfoInterface* profile_cache,
+AvatarMenu::AvatarMenu(ProfileAttributesStorage* /*profile_storage*/,

[lwchkg, 2016/02/14 17:07:18]

Currently only the interface of AvatarMenu refactored, and that's why this
variable is unused. The actual refactoring is too large to be included in this
CL, and I'll work on that later.

[lwchkg, 2016/02/14 17:22:37]

I've found another 5 Mac-only files calling this constructor. However, uploading
these files requires adding more reviewers. If the try-bots pass (which should)
without modifying these 5 files, I'll refactor these files in other CLs.

                        AvatarMenuObserver* observer,
                        Browser* browser)
-    : profile_list_(ProfileList::Create(profile_cache)),
+    : profile_list_(ProfileList::Create(
+          &g_browser_process->profile_manager()->GetProfileInfoCache())),
       menu_actions_(AvatarMenuActions::Create()),
 #if defined(ENABLE_SUPERVISED_USERS)
       supervised_user_observer_(this),
 #endif
-      profile_info_(profile_cache),
+      profile_info_(
+          &g_browser_process->profile_manager()->GetProfileInfoCache()),
       observer_(observer),
       browser_(browser) {
   DCHECK(profile_info_);