Utilize ProfileInfoCache to support data type counts in profile deletion flow

chrome/browser/profiles/profile_statistics.cc

 // Copyright 2015 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_statistics.h"
 
+#include <set>
 #include "base/bind.h"
 #include "base/memory/ref_counted.h"
 #include "base/prefs/pref_service.h"
 #include "base/task_runner.h"
 #include "base/time/time.h"
 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/history/history_service_factory.h"
 #include "chrome/browser/password_manager/password_store_factory.h"
 #include "chrome/browser/profiles/profile_attributes_entry.h"
 #include "chrome/browser/profiles/profile_attributes_storage.h"
 #include "chrome/browser/profiles/profile_info_cache.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "components/bookmarks/browser/bookmark_model.h"
+#include "components/bookmarks/browser/bookmark_model_observer.h"
 #include "components/history/core/browser/history_service.h"
 #include "components/password_manager/core/browser/password_store.h"
 #include "components/password_manager/core/browser/password_store_consumer.h"
 #include "content/public/browser/browser_thread.h"
 
+using bookmarks::BookmarkModel;

[Mike Lerman, 2015/11/26 15:06:25]

ugh; please try not to use using statements.

[lwchkg, 2015/12/02 15:33:34]

Acknowledged.

+using bookmarks::BookmarkNode;
 using content::BrowserThread;
 
 namespace {
 
 struct ProfileStatValue {
   int count;
   bool success;  // false means the statistics failed to load
 };
 
-int CountBookmarksFromNode(const bookmarks::BookmarkNode* node) {
+int CountBookmarksFromNode(const BookmarkNode* node) {
   int count = 0;
   if (node->is_url()) {
     ++count;
   } else {
     for (int i = 0; i < node->child_count(); ++i)
       count += CountBookmarksFromNode(node->GetChild(i));
   }
   return count;
 }
 
 class ProfileStatisticsAggregator
     : public base::RefCountedThreadSafe<ProfileStatisticsAggregator> {
-  // This class collects statistical information about the profile and returns
+  // This class is used internally by GetProfileStatistics and
+  // StoreProfileStatisticsToCache.
+  //
+  // The class collects statistical information about the profile and returns
   // the information via a callback function. Currently bookmarks, history,
   // logins and preferences are counted.
   //
   // The class is RefCounted because this is needed for CancelableTaskTracker
   // to function properly. Once all tasks are run (or cancelled) the instance is
   // automatically destructed.
-  //
-  // The class is used internally by GetProfileStatistics function.
 
  public:
-  explicit ProfileStatisticsAggregator(Profile* profile,
+  ProfileStatisticsAggregator(Profile* profile,
       const profiles::ProfileStatisticsCallback& callback,
       base::CancelableTaskTracker* tracker);
 
  private:
   friend class base::RefCountedThreadSafe<ProfileStatisticsAggregator>;
   ~ProfileStatisticsAggregator() {}
 
   void Init();
 
   // Callback functions
   // Normal callback. Appends result to |profile_category_stats_|, and then call
   // the external callback. All other callbacks call this function.
   void StatisticsCallback(const char* category, ProfileStatValue result);
   // Callback for reporting success.
   void StatisticsCallbackSuccess(const char* category, int count);
   // Callback for reporting failure.
   void StatisticsCallbackFailure(const char* category);
   // Callback for history.
   void StatisticsCallbackHistory(history::HistoryCountResult result);
 
   // Bookmark counting.
-  ProfileStatValue CountBookmarks() const;
+  void PrepareCountBookmarks();

[Mike Lerman, 2015/11/26 15:06:25]

Perhaps call this WaitOrCountBookmarks()? The method doesn't really do any
preparation - it just waits.

[lwchkg, 2015/12/02 15:33:34]

Good idea. Thanks!

+  void CountBookmarks(BookmarkModel* bookmark_model);
 
   // Preference counting.
   ProfileStatValue CountPrefs() const;
 
   Profile* profile_;
   profiles::ProfileCategoryStats profile_category_stats_;
 
   // Callback function to be called when results arrive. Will be called
   // multiple times (once for each statistics).
   const profiles::ProfileStatisticsCallback callback_;
 
   base::CancelableTaskTracker* tracker_;
+  scoped_ptr<base::CancelableTaskTracker> default_tracker_;
+
+  // Bookmark counting
+  class BookmarkModelHelper
+      : public bookmarks::BookmarkModelObserver {
+   public:
+    explicit BookmarkModelHelper(ProfileStatisticsAggregator* parent)
+        : parent_(parent) {}
+
+    void BookmarkModelLoaded(BookmarkModel* model, bool ids_reassigned)
+        override {
+      // Remove observer before release, otherwise it may become a dangling
+      // reference.
+      model->RemoveObserver(this);
+      parent_->CountBookmarks(model);
+      parent_->Release();
+    }
+
+    void BookmarkNodeMoved(BookmarkModel* model, const BookmarkNode* old_parent,
+                           int old_index, const BookmarkNode* new_parent,
+                           int new_index) override {}
+
+    void BookmarkNodeAdded(BookmarkModel* model, const BookmarkNode* parent,
+                           int index) override {}
+
+    void BookmarkNodeRemoved(BookmarkModel* model, const BookmarkNode* parent,
+        int old_index, const BookmarkNode* node,
+        const std::set<GURL>& no_longer_bookmarked) override {}
+
+    void BookmarkNodeChanged(BookmarkModel* model, const BookmarkNode* node)
+        override {}
+
+    void BookmarkNodeFaviconChanged(BookmarkModel* model,
+                                    const BookmarkNode* node) override {}
+
+    void BookmarkNodeChildrenReordered(BookmarkModel* model,
+                                       const BookmarkNode* node) override {}
+
+    void BookmarkAllUserNodesRemoved(BookmarkModel* model,
+        const std::set<GURL>& removed_urls) override {}
+
+
+   private:
+    ProfileStatisticsAggregator* parent_ = nullptr;
+  };
+  scoped_ptr<BookmarkModelHelper> bookmark_model_helper_;
 
   // Password counting.
   class PasswordStoreConsumerHelper
       : public password_manager::PasswordStoreConsumer {
    public:
     explicit PasswordStoreConsumerHelper(ProfileStatisticsAggregator* parent)
         : parent_(parent) {}
 
     void OnGetPasswordStoreResults(
         ScopedVector<autofill::PasswordForm> results) override {
@@ skipping 12 matching lines @@
 };
 
 ProfileStatisticsAggregator::ProfileStatisticsAggregator(
     Profile* profile,
     const profiles::ProfileStatisticsCallback& callback,
     base::CancelableTaskTracker* tracker)
     : profile_(profile),
       callback_(callback),
       tracker_(tracker),
       password_store_consumer_helper_(this) {
+  if (!tracker_) {
+    default_tracker_.reset(new base::CancelableTaskTracker);
+    tracker_ = default_tracker_.get();
+  }
   Init();
 }
 
 void ProfileStatisticsAggregator::Init() {
+  DCHECK(profile_);
+
   // Initiate bookmark counting (async). Post to UI thread.
-  tracker_->PostTaskAndReplyWithResult(
+  tracker_->PostTask(
       BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI).get(),
-          FROM_HERE,
-          base::Bind(&ProfileStatisticsAggregator::CountBookmarks, this),
-          base::Bind(&ProfileStatisticsAggregator::StatisticsCallback,
-                     this, profiles::kProfileStatisticsBookmarks));
+      FROM_HERE,
+      base::Bind(&ProfileStatisticsAggregator::PrepareCountBookmarks, this));
 
   // Initiate history counting (async).
   history::HistoryService* history_service =
       HistoryServiceFactory::GetForProfileWithoutCreating(profile_);
 
   if (history_service) {
     history_service->GetHistoryCount(
         base::Time(),
         base::Time::Max(),
         base::Bind(&ProfileStatisticsAggregator::StatisticsCallbackHistory,
@@ skipping 10 matching lines @@
           profile_, ServiceAccessType::EXPLICIT_ACCESS);
   if (password_store) {
     password_store->GetAutofillableLogins(&password_store_consumer_helper_);
   } else {
     StatisticsCallbackFailure(profiles::kProfileStatisticsPasswords);
   }
 
   // Initiate preference counting (async). Post to UI thread.
   tracker_->PostTaskAndReplyWithResult(
       BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI).get(),
-          FROM_HERE,
-          base::Bind(&ProfileStatisticsAggregator::CountPrefs, this),
-          base::Bind(&ProfileStatisticsAggregator::StatisticsCallback,
-                     this, profiles::kProfileStatisticsSettings));
+      FROM_HERE,
+      base::Bind(&ProfileStatisticsAggregator::CountPrefs, this),
+      base::Bind(&ProfileStatisticsAggregator::StatisticsCallback,
+                 this, profiles::kProfileStatisticsSettings));
 }
 
 void ProfileStatisticsAggregator::StatisticsCallback(
     const char* category, ProfileStatValue result) {
   profiles::ProfileCategoryStat datum;
   datum.category = category;
   datum.count = result.count;
   datum.success = result.success;
   profile_category_stats_.push_back(datum);
-  callback_.Run(profile_category_stats_);
+  if (!callback_.is_null())
+    callback_.Run(profile_category_stats_);
+
+  profiles::SetProfileStatisticsInCache(profile_->GetPath(), datum.category,
+                                        result.count);
 }
 
 void ProfileStatisticsAggregator::StatisticsCallbackSuccess(
     const char* category, int count) {
   ProfileStatValue result;
   result.count = count;
   result.success = true;
   StatisticsCallback(category, result);
 }
 
 void ProfileStatisticsAggregator::StatisticsCallbackFailure(
     const char* category) {
   ProfileStatValue result;
   result.count = 0;
   result.success = false;
   StatisticsCallback(category, result);
 }
 
 void ProfileStatisticsAggregator::StatisticsCallbackHistory(
     history::HistoryCountResult result) {
   ProfileStatValue result_converted;
   result_converted.count = result.count;
   result_converted.success = result.success;
   StatisticsCallback(profiles::kProfileStatisticsBrowsingHistory,
                      result_converted);
 }
 
-ProfileStatValue ProfileStatisticsAggregator::CountBookmarks() const {
-  bookmarks::BookmarkModel* bookmark_model =
+void ProfileStatisticsAggregator::CountBookmarks(
+    BookmarkModel* bookmark_model) {
+  int count = CountBookmarksFromNode(bookmark_model->bookmark_bar_node()) +
+              CountBookmarksFromNode(bookmark_model->other_node()) +
+              CountBookmarksFromNode(bookmark_model->mobile_node());
+
+  StatisticsCallbackSuccess(profiles::kProfileStatisticsBookmarks, count);
+}
+
+void ProfileStatisticsAggregator::PrepareCountBookmarks() {
+  BookmarkModel* bookmark_model =
       BookmarkModelFactory::GetForProfileIfExists(profile_);
 
-  ProfileStatValue result;
   if (bookmark_model) {
-    result.count = CountBookmarksFromNode(bookmark_model->bookmark_bar_node()) +
-                   CountBookmarksFromNode(bookmark_model->other_node()) +
-                   CountBookmarksFromNode(bookmark_model->mobile_node());
-    result.success = true;
+    if (bookmark_model->loaded()) {
+      CountBookmarks(bookmark_model);
+    } else {
+      AddRef();
+      bookmark_model_helper_.reset(new BookmarkModelHelper(this));
+      bookmark_model->AddObserver(bookmark_model_helper_.get());
+    }
   } else {
-    result.count = 0;
-    result.success = false;
+    StatisticsCallbackFailure(profiles::kProfileStatisticsBookmarks);
   }

[Mike Lerman, 2015/11/26 15:06:25]

if there's no bookmark_model, what's the appropriate result?

[lwchkg, 2015/12/02 15:33:34]

The statistics should fail. That's what StatisticsCallbackFailure is doing.

-  return result;
 }
 
 ProfileStatValue ProfileStatisticsAggregator::CountPrefs() const {
   const PrefService* pref_service = profile_->GetPrefs();
 
   ProfileStatValue result;
   if (pref_service) {
     scoped_ptr<base::DictionaryValue> prefs =
         pref_service->GetPreferenceValuesWithoutPathExpansion();
 
@@ skipping 28 matching lines @@
 const char kProfileStatisticsBookmarks[] = "Bookmarks";
 const char kProfileStatisticsSettings[] = "Settings";
 
 void GetProfileStatistics(Profile* profile,
     const ProfileStatisticsCallback& callback,
     base::CancelableTaskTracker* tracker) {
   scoped_refptr<ProfileStatisticsAggregator> aggregator =
       new ProfileStatisticsAggregator(profile, callback, tracker);
 }
 
+void StoreProfileStatisticsToCache(Profile* profile) {
+  GetProfileStatistics(profile, profiles::ProfileStatisticsCallback(), nullptr);
+}
+
 ProfileCategoryStats GetProfileStatisticsFromCache(
     const base::FilePath& profile_path) {
   ProfileInfoCache& profile_info_cache =
       g_browser_process->profile_manager()->GetProfileInfoCache();
   ProfileAttributesEntry* entry = nullptr;
   bool has_entry = profile_info_cache.
       GetProfileAttributesWithPath(profile_path, &entry);
 
   ProfileCategoryStats stats;
   ProfileCategoryStat stat;
@@ skipping 40 matching lines @@
   } else if (category == kProfileStatisticsBookmarks) {
     entry->SetStatsBookmarks(count);
   } else if (category == kProfileStatisticsSettings) {
     entry->SetStatsSettings(count);
   } else {
     NOTREACHED();
   }
 }
 
 }  // namespace profiles