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 kStatistics[] = "statistics";
 
 // 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::GetStatisticOfProfileAtIndex(size_t index,
+    const std::string& category, int* out_value) const {
+  const base::DictionaryValue* statistics;
+  if (!GetInfoForProfileAtIndex(index)->GetDictionaryWithoutPathExpansion(
+      kStatistics, &statistics))
+    return false;
+
+  return statistics->GetIntegerWithoutPathExpansion(category, out_value);
+}
+
+std::map<std::string, int> ProfileInfoCache::GetAllStatisticsOfProfileAtIndex(
+    size_t index) const {
+  const base::DictionaryValue* statistics;
+  std::map<std::string, int> out_value;
+
+  if (!GetInfoForProfileAtIndex(index)->GetDictionaryWithoutPathExpansion(
+      kStatistics, &statistics))
+    return out_value;  // No statistics are set, so return an empty map.
+
+  for (base::DictionaryValue::Iterator it(*statistics); !it.IsAtEnd();
+       it.Advance()) {
+    int value;
+    if (it.value().GetAsInteger(&value)) {
+      out_value[it.key()] = value;
+    }
+  }
+  return out_value;
+}
+
+scoped_ptr<base::DictionaryValue>
+    ProfileInfoCache::GetAllStatisticsOfProfileAtIndexAsDictionaryValue(
+    size_t index) const {
+  const base::DictionaryValue* statistics;
+
+  if (!GetInfoForProfileAtIndex(index)->GetDictionaryWithoutPathExpansion(
+      kStatistics, &statistics))
+    // No statistics are set, so return an empty map.
+    return make_scoped_ptr(new base::DictionaryValue()).Pass();
+
+  return statistics->CreateDeepCopy().Pass();
+}
+
 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::SetStatisticOfProfileAtIndex(size_t index,
+    const std::string& category, int value) {
+  scoped_ptr<base::DictionaryValue> info(
+      GetInfoForProfileAtIndex(index)->DeepCopy());
+  base::DictionaryValue* statistics;
+  if (!info->GetDictionaryWithoutPathExpansion(kStatistics, &statistics)) {
+    statistics = new base::DictionaryValue();
+    info->SetWithoutPathExpansion(kStatistics, make_scoped_ptr(statistics));
+  }
+
+  statistics->SetIntegerWithoutPathExpansion(category, value);
+  // This takes ownership of |info|.
+  SetInfoForProfileAtIndex(index, info.release());
+
+  base::FilePath profile_path = GetPathOfProfileAtIndex(index);
+  FOR_EACH_OBSERVER(ProfileInfoCacheObserver,

[Mike Lerman, 2015/10/23 16:04:19]

Why does changing the stats set OnProfileSigninRequiredChanged?

[lwchkg, 2015/10/24 02:02:32]

Thanks for catching the mistake. We don't need an observer here, so I'll remove
these lines.

+                    observer_list_,
+                    OnProfileSigninRequiredChanged(profile_path));
+}
+
 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;
 }