Add counts of User data to ProfileInfoCache

chrome/browser/profiles/profile_info_cache.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/profiles/profile_info_cache.h"
 
 #include "base/bind.h"
 #include "base/files/file_util.h"
 #include "base/i18n/case_conversion.h"
 #include "base/logging.h"
@@ skipping 42 matching lines @@
 const char kPasswordTokenKey[] = "gaia_password_token";
 const char kUseGAIAPictureKey[] = "use_gaia_picture";
 const char kBackgroundAppsKey[] = "background_apps";
 const char kGAIAPictureFileNameKey[] = "gaia_picture_file_name";
 const char kIsOmittedFromProfileListKey[] = "is_omitted_from_profile_list";
 const char kSigninRequiredKey[] = "signin_required";
 const char kSupervisedUserId[] = "managed_user_id";
 const char kProfileIsEphemeral[] = "is_ephemeral";
 const char kActiveTimeKey[] = "active_time";
 const char kIsAuthErrorKey[] = "is_auth_error";
+const char kStatsBrowsingHistoryKey[] = "stats_browsing_history";
+const char kStatsPasswordsKey[] = "stats_passwords";
+const char kStatsBookmarksKey[] = "stats_bookmarks";
+const char kStatsSettingsKey[] = "stats_settings";
 
 // First eight are generic icons, which use IDS_NUMBERED_PROFILE_NAME.
 const int kDefaultNames[] = {
   IDS_DEFAULT_AVATAR_NAME_8,
   IDS_DEFAULT_AVATAR_NAME_9,
   IDS_DEFAULT_AVATAR_NAME_10,
   IDS_DEFAULT_AVATAR_NAME_11,
   IDS_DEFAULT_AVATAR_NAME_12,
   IDS_DEFAULT_AVATAR_NAME_13,
   IDS_DEFAULT_AVATAR_NAME_14,
@@ skipping 419 matching lines @@
     const {
   std::string icon_url;
   GetInfoForProfileAtIndex(index)->GetString(kAvatarIconKey, &icon_url);
   size_t icon_index = 0;
   if (!profiles::IsDefaultAvatarIconUrl(icon_url, &icon_index))
     DLOG(WARNING) << "Unknown avatar icon: " << icon_url;
 
   return icon_index;
 }
 
+bool ProfileInfoCache::HasStatsBrowsingHistoryOfProfileAtIndex(size_t index)
+    const {
+  int value = 0;
+  return GetInfoForProfileAtIndex(index)->GetInteger(kStatsBrowsingHistoryKey,
+                                                     &value);
+}
+
+int ProfileInfoCache::GetStatsBrowsingHistoryOfProfileAtIndex(size_t index)
+    const {
+  int value = 0;
+  GetInfoForProfileAtIndex(index)->GetInteger(kStatsBrowsingHistoryKey, &value);
+  return value;
+}
+
+bool ProfileInfoCache::HasStatsPasswordsOfProfileAtIndex(size_t index)
+    const {
+  int value = 0;
+  return GetInfoForProfileAtIndex(index)->GetInteger(kStatsPasswordsKey,
+                                                     &value);
+}
+
+int ProfileInfoCache::GetStatsPasswordsOfProfileAtIndex(size_t index)
+    const {
+  int value = 0;
+  GetInfoForProfileAtIndex(index)->GetInteger(kStatsPasswordsKey, &value);
+  return value;
+}
+
+bool ProfileInfoCache::HasStatsBookmarksOfProfileAtIndex(size_t index)
+    const {
+  int value = 0;
+  return GetInfoForProfileAtIndex(index)->GetInteger(kStatsBookmarksKey,
+                                                     &value);
+}
+
+int ProfileInfoCache::GetStatsBookmarksOfProfileAtIndex(size_t index)
+    const {
+  int value = 0;
+  GetInfoForProfileAtIndex(index)->GetInteger(kStatsBookmarksKey, &value);
+  return value;
+}
+
+bool ProfileInfoCache::HasStatsSettingsOfProfileAtIndex(size_t index)

[Mike Lerman, 2015/11/17 16:29:51]

nit: I think const can go on the line before. (and perhaps in a couple of the
others)

[lwchkg, 2015/11/18 17:34:53]

Done. Six pairs replaced.

+    const {
+  int value = 0;
+  return GetInfoForProfileAtIndex(index)->GetInteger(kStatsSettingsKey, &value);
+}
+
+int ProfileInfoCache::GetStatsSettingsOfProfileAtIndex(size_t index)
+    const {
+  int value = 0;
+  GetInfoForProfileAtIndex(index)->GetInteger(kStatsSettingsKey, &value);
+  return value;
+}
+
 void ProfileInfoCache::SetProfileActiveTimeAtIndex(size_t index) {
   if (base::Time::Now() - GetProfileActiveTimeAtIndex(index) <
       base::TimeDelta::FromHours(1)) {
     return;
   }
 
   scoped_ptr<base::DictionaryValue> info(
       GetInfoForProfileAtIndex(index)->DeepCopy());
   info->SetDouble(kActiveTimeKey, base::Time::Now().ToDoubleT());
   // This takes ownership of |info|.
@@ skipping 390 matching lines @@
   if (ChooseAvatarIconIndexForNewProfile(true, true, &icon_index))
     return icon_index;
   // Settle for any random icon, even if it's not unique.
   if (ChooseAvatarIconIndexForNewProfile(true, false, &icon_index))
     return icon_index;
 
   NOTREACHED();
   return 0;
 }
 
+void ProfileInfoCache::SetStatsBrowsingHistoryOfProfileAtIndex(size_t index,
+                                                               int value) {
+  scoped_ptr<base::DictionaryValue> info(
+      GetInfoForProfileAtIndex(index)->DeepCopy());
+  info->SetInteger(kStatsBrowsingHistoryKey, value);
+  // This takes ownership of |info|.
+  SetInfoForProfileAtIndex(index, info.release());
+}
+
+void ProfileInfoCache::SetStatsPasswordsOfProfileAtIndex(size_t index,
+                                                               int value) {

[anthonyvd, 2015/11/17 19:03:30]

nit: alignment of the params here and the following 2 functions.

[lwchkg, 2015/11/18 17:34:53]

Done. (Anyway, what's happening with me with those alignments? Embarrassing.)

+  scoped_ptr<base::DictionaryValue> info(
+      GetInfoForProfileAtIndex(index)->DeepCopy());
+  info->SetInteger(kStatsPasswordsKey, value);
+  // This takes ownership of |info|.
+  SetInfoForProfileAtIndex(index, info.release());
+}
+
+void ProfileInfoCache::SetStatsBookmarksOfProfileAtIndex(size_t index,
+                                                               int value) {
+  scoped_ptr<base::DictionaryValue> info(
+      GetInfoForProfileAtIndex(index)->DeepCopy());
+  info->SetInteger(kStatsBookmarksKey, value);
+  // This takes ownership of |info|.
+  SetInfoForProfileAtIndex(index, info.release());
+}
+
+void ProfileInfoCache::SetStatsSettingsOfProfileAtIndex(size_t index,
+                                                               int value) {
+  scoped_ptr<base::DictionaryValue> info(
+      GetInfoForProfileAtIndex(index)->DeepCopy());
+  info->SetInteger(kStatsSettingsKey, value);
+  // This takes ownership of |info|.
+  SetInfoForProfileAtIndex(index, info.release());
+}
+
 const base::FilePath& ProfileInfoCache::GetUserDataDir() const {
   return user_data_dir_;
 }
 
 // static
 void ProfileInfoCache::RegisterPrefs(PrefRegistrySimple* registry) {
   registry->RegisterDictionaryPref(prefs::kProfileInfoCache);
 }
 
 void ProfileInfoCache::DownloadHighResAvatarIfNeeded(
@@ skipping 359 matching lines @@
     // The profile info is in the cache but its entry isn't created yet, insert
     // it in the map.
     scoped_ptr<ProfileAttributesEntry> new_entry(new ProfileAttributesEntry());
     profile_attributes_entries_.add(path, new_entry.Pass());
     profile_attributes_entries_.get(path)->Initialize(this, path);
   }
 
   *entry = profile_attributes_entries_.get(path);
   return true;
 }