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

Unified Diff: components/password_manager/core/browser/login_database.h

Issue 355143006: [Password Manager] Add UMA stats to track interaction with synced accounts. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Comments Created 6 years, 6 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: components/password_manager/core/browser/login_database.h
diff --git a/components/password_manager/core/browser/login_database.h b/components/password_manager/core/browser/login_database.h
index ac6bd0be6349d353d8421c7854e40c7a197d4b7f..e964a148411cda9371ea9a2feca7d11c52c5a208 100644
--- a/components/password_manager/core/browser/login_database.h
+++ b/components/password_manager/core/browser/login_database.h
@@ -13,9 +13,13 @@
#include "base/strings/string16.h"
#include "components/password_manager/core/browser/password_store_change.h"
#include "components/password_manager/core/browser/psl_matching_helper.h"
+#include "components/password_manager/core/common/sync_state.h"
#include "sql/connection.h"
#include "sql/meta_table.h"
+class ProfileSyncService;
+class SigninManagerBase;
+
namespace password_manager {
// Interface to the database storage of login information, intended as a helper
@@ -31,7 +35,8 @@ class LoginDatabase {
bool Init(const base::FilePath& db_path);
// Reports usage metrics to UMA.
- void ReportMetrics();
+ void ReportMetrics(const std::string& sync_username,
+ password_manager::SyncState sync_state);
// Adds |form| to the list of remembered password forms. Returns the list of
// changes applied ({}, {ADD}, {REMOVE, ADD}). If it returns {REMOVE, ADD}

Powered by Google App Engine
This is Rietveld 408576698