Sooper experimental refactoring of the profile info cache.

chrome/browser/ui/app_list/app_list_view_delegate.cc

 // Copyright (c) 2012 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/ui/app_list/app_list_view_delegate.h"
 
 #include <vector>
 
 #include "base/callback.h"
 #include "base/files/file_path.h"
 #include "base/stl_util.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/feedback/feedback_util.h"
 #include "chrome/browser/profiles/profile_info_cache.h"
+#include "chrome/browser/profiles/profile_info_entry.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/app_list/app_list_controller_delegate.h"
 #include "chrome/browser/ui/app_list/extension_app_model_builder.h"
 #include "chrome/browser/ui/app_list/search/search_controller.h"
 #include "chrome/browser/ui/app_list/start_page_service.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/chrome_pages.h"
 #include "chrome/browser/ui/host_desktop.h"
 #include "chrome/browser/ui/web_applications/web_app_ui.h"
 #include "chrome/browser/web_applications/web_app.h"
@@ skipping 25 matching lines @@
       content::BrowserThread::FILE,
       FROM_HERE,
       base::Bind(web_app::CreateShortcutInWebAppDir, app_data_dir, info),
       callback);
 }
 #endif
 
 void PopulateUsers(const ProfileInfoCache& profile_info,
                    const base::FilePath& active_profile_path,
                    app_list::AppListModel::Users* users) {
-  const size_t count = profile_info.GetNumberOfProfiles();
-  for (size_t i = 0; i < count; ++i) {
-    // Don't display managed users.
-    if (profile_info.ProfileIsManagedAtIndex(i))
+
+  const std::vector<ProfileInfoEntry> entries(
+      profile_info.GetProfilesSortedByName());
+
+  for (std::vector<ProfileInfoEntry>::const_iterator it = entries.begin();
+      it != entries.end(); ++it) {
+    if (it->IsManaged())
       continue;
 
     app_list::AppListModel::User user;
-    user.name = profile_info.GetNameOfProfileAtIndex(i);
-    user.email = profile_info.GetUserNameOfProfileAtIndex(i);
-    user.profile_path = profile_info.GetPathOfProfileAtIndex(i);
+    user.name = it->GetDisplayName();
+    user.email = it->user_name();
+    user.profile_path = it->path();
     user.active = active_profile_path == user.profile_path;
     users->push_back(user);
   }
 }
 
 }  // namespace
 
 AppListViewDelegate::AppListViewDelegate(
     scoped_ptr<AppListControllerDelegate> controller,
     Profile* profile)
@@ skipping 207 matching lines @@
 }
 
 content::WebContents* AppListViewDelegate::GetStartPageContents() {
   app_list::StartPageService* service =
       app_list::StartPageService::Get(profile_);
   if (!service)
     return NULL;
 
   return service->contents();
 }