Sooper experimental refactoring of the profile info cache.

chrome/browser/notifications/message_center_settings_controller.cc

 // Copyright (c) 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/notifications/message_center_settings_controller.h"
 
 #include <algorithm>
 
 #include "base/command_line.h"
 #include "base/i18n/string_compare.h"
@@ skipping 363 matching lines @@
 #else
   NOTREACHED();
   return NULL;
 #endif
 }
 
 void MessageCenterSettingsController::RebuildNotifierGroups() {
   notifier_groups_.clear();
   current_notifier_group_ = 0;
 
-  const size_t count = profile_info_cache_->GetNumberOfProfiles();
-
-  for (size_t i = 0; i < count; ++i) {
+  const std::vector<ProfileInfoEntry> entries(
+      profile_info_cache_->GetProfilesSortedByName());
+  int index = 0;
+  for (std::vector<ProfileInfoEntry>::const_iterator it = entries.begin();
+    it != entries.end(); ++it) {
     scoped_ptr<message_center::ProfileNotifierGroup> group(
         new message_center::ProfileNotifierGroup(
-            profile_info_cache_->GetAvatarIconOfProfileAtIndex(i),
-            profile_info_cache_->GetNameOfProfileAtIndex(i),
-            profile_info_cache_->GetUserNameOfProfileAtIndex(i),
-            i,
-            profile_info_cache_->GetPathOfProfileAtIndex(i)));
+            profile_info_cache_->GetAvatarIconOfProfile(it->path()),
+            it->GetDisplayName(),
+            it->user_name(),
+            index++,
+            it->path()));
     if (group->profile() == NULL)
       continue;
 
 #if defined(OS_CHROMEOS)
     // In ChromeOS, the login screen first creates a dummy profile which is not
     // actually used, and then the real profile for the user is created when
     // login (or turns into kiosk mode). This profile should be skipped.
     if (chromeos::ProfileHelper::IsSigninProfile(group->profile()))
       continue;
 #endif
     notifier_groups_.push_back(group.release());
   }
 }