Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4848)

Unified Diff: chrome/browser/profiles/profile_statistics.h

Issue 1248613003: Issue 501916 : Add data type counts to profile deletion flow (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Third draft - removed all password manager code Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/profiles/profile_statistics.h
diff --git a/chrome/browser/profiles/profile_statistics.h b/chrome/browser/profiles/profile_statistics.h
new file mode 100644
index 0000000000000000000000000000000000000000..2a5fd39ff1658815c60a1f74447fb6c59bd8bbca
--- /dev/null
+++ b/chrome/browser/profiles/profile_statistics.h
@@ -0,0 +1,34 @@
+// 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.
+
+#ifndef CHROME_BROWSER_PROFILES_PROFILE_STATISTICS_H_
+#define CHROME_BROWSER_PROFILES_PROFILE_STATISTICS_H_
+
+#include <string>
+#include <utility>
+#include <vector>
+#include "base/task/cancelable_task_tracker.h"
+#include "chrome/browser/profiles/profile.h"
+
+namespace profiles {
msramek 2015/07/29 18:10:25 Nit: Empty line after this line.
lwchkg 2015/07/30 17:07:19 Acknowledged.
+// Definition of the return value of |ProfileStatisticsCallback|.
+// The data stored in |ProfileStatisticsValues| looks like this:
+// <"Browsing History", 912>, <"Passwords", 71>, <"Bookmarks", 120>,
+// <"Settings", 200>
+typedef std::vector<std::pair<std::string, int> > ProfileStatisticsValues;
msramek 2015/07/29 18:10:25 Wouldn't std::map<std::string, int> be preferable?
lwchkg 2015/07/30 17:07:19 I agree, but that vector is the idea of Mike, so I
+// Definition of the callback function. Note that a copy of
+// |ProfileStatisticsValues| is made each time the callback is called.
+typedef base::Callback<void(ProfileStatisticsValues)>
+ ProfileStatisticsCallback;
+
+// This class collect statistical information about the profile and returns the
msramek 2015/07/29 18:10:25 Copy-pasted comment. Please update.
lwchkg 2015/07/30 17:07:19 Acknowledged.
+// information via a callback function. Currently bookmarks, history, logins and
+// preferences are counted. The callback function may be called more than once.
+void GetProfileStatistics(Profile* profile,
msramek 2015/07/29 18:10:25 Nit: formatting. http://google-styleguide.googlec
lwchkg 2015/07/30 17:07:19 Acknowledged.
+ const ProfileStatisticsCallback& callback,
+ base::CancelableTaskTracker* tracker);
+
+} // namespace profiles
+
+#endif // CHROME_BROWSER_PROFILES_PROFILE_STATISTICS_H_

Powered by Google App Engine
This is Rietveld 408576698