Sooper experimental refactoring of the profile info cache.

chrome/browser/profiles/profile_list_desktop.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/profiles/profile_list_desktop.h"
 
 #include "base/command_line.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_info_cache.h"
 #include "chrome/browser/profiles/profile_info_util.h"
@@ skipping 16 matching lines @@
 
 size_t ProfileListDesktop::GetNumberOfItems() const {
   return items_.size();
 }
 
 const AvatarMenu::Item& ProfileListDesktop::GetItemAt(size_t index) const {
   DCHECK_LT(index, items_.size());
   return *items_[index];
 }
 
+const AvatarMenu::Item& ProfileListDesktop::GetActiveItem() const {
+  for (size_t i = 0; i < items_.size(); ++i) {
+    if (items_[i]->active)
+      return *items_[i];
+  }
+
+  // During singleton profile deletion, this function can be called with no
+  // profiles in the model - crbug.com/102278 .
+  return *items_[0];
+}
+
 void ProfileListDesktop::RebuildMenu() {
   ClearMenu();
 
-  const size_t count = profile_info_->GetNumberOfProfiles();
-  for (size_t i = 0; i < count; ++i) {
+  const std::vector<ProfileInfoEntry> entries(
+      profile_info_->GetProfilesSortedByName());
+  for (size_t i = 0; i < entries.size(); ++i) {
+    const ProfileInfoEntry& entry = entries[i];
     bool is_gaia_picture =
-        profile_info_->IsUsingGAIAPictureOfProfileAtIndex(i) &&
-        profile_info_->GetGAIAPictureOfProfileAtIndex(i);
+        entry.is_using_GAIA_picture() &&
+        profile_info_->GetGAIAPictureOfProfile(entry.path());
 
-    gfx::Image icon = profile_info_->GetAvatarIconOfProfileAtIndex(i);
+    gfx::Image icon = profile_info_->GetAvatarIconOfProfile(entry.path());
     if (!CommandLine::ForCurrentProcess()->HasSwitch(
         switches::kNewProfileManagement)) {
       // old avatar menu uses resized-small images
       icon = profiles::GetAvatarIconForMenu(icon, is_gaia_picture);
     }
 
     AvatarMenu::Item* item = new AvatarMenu::Item(i, i, icon);
-    item->name = profile_info_->GetNameOfProfileAtIndex(i);
-    item->sync_state = profile_info_->GetUserNameOfProfileAtIndex(i);
-    item->managed = profile_info_->ProfileIsManagedAtIndex(i);
+    item->name = entry.GetDisplayName();
+    item->sync_state = entry.user_name();
+    item->managed = entry.IsManaged();
     item->signed_in = !item->sync_state.empty();
     if (!item->signed_in) {
       item->sync_state = l10n_util::GetStringUTF16(
           item->managed ? IDS_MANAGED_USER_AVATAR_LABEL :
                           IDS_PROFILES_LOCAL_PROFILE_STATE);
     }
-    item->active = profile_info_->GetPathOfProfileAtIndex(i) ==
-        active_profile_path_;
-    item->signin_required = profile_info_->ProfileIsSigninRequiredAtIndex(i);
+    item->active = entry.path() == active_profile_path_;
+    item->signin_required = entry.is_signin_required();
     items_.push_back(item);
   }
 }
 
 size_t ProfileListDesktop::MenuIndexFromProfileIndex(size_t index) {
   // Menu indices correspond to indices in profile cache.
   return index;
 }
 
 void ProfileListDesktop::ActiveProfilePathChanged(base::FilePath& path) {
   active_profile_path_ = path;
 }
 
 void ProfileListDesktop::ClearMenu() {
   STLDeleteElements(&items_);
 }