Refactor most c/b/profiles calls to ProfileInfoCache.

chrome/browser/profiles/profile_attributes_entry.h

Index: chrome/browser/profiles/profile_attributes_entry.h
diff --git a/chrome/browser/profiles/profile_attributes_entry.h b/chrome/browser/profiles/profile_attributes_entry.h
index 898557792594d675337a446ae23b823c990f8b63..c2f12d0d002370d45b8263b1144d38aba671e19e 100644
--- a/chrome/browser/profiles/profile_attributes_entry.h
+++ b/chrome/browser/profiles/profile_attributes_entry.h
@@ -86,6 +86,7 @@ class ProfileAttributesEntry {
 
   void SetName(const base::string16& name);
   void SetShortcutName(const base::string16& name);
+  void SetActiveTime();
   void SetIsOmitted(bool is_omitted);
   void SetSupervisedUserId(const std::string& id);
   void SetLocalAuthCredentials(const std::string& auth);
@@ -105,6 +106,8 @@ class ProfileAttributesEntry {
   void SetAuthInfo(const std::string& gaia_id,
                    const base::string16& user_name);
 
+  bool LessThan(const ProfileAttributesEntry& other) const;

[Mike Lerman, 2015/08/06 16:06:19]

Add a comment here? It's unclear how we're comparing ProfileAttributeEntries.
(who needs this? Does this really belong here? I don't think "name" is necessary
the canonical way to compare profiles for all time, but maybe it is)

+
  private:
   // These members are an implementation detail meant to smooth the migration
   // of the ProfileInfoCache to the ProfileAttributesStorage interface. They can