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

Unified Diff: chrome/browser/password_manager/password_store_mac.cc

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: Typo Created 6 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/password_manager/password_store_mac.cc
diff --git a/chrome/browser/password_manager/password_store_mac.cc b/chrome/browser/password_manager/password_store_mac.cc
index 63359dbbc5576ee7560519e4dc25ff57d490b581..daf9fb9dc5e6b2f57c0e585bbc505f28fdcec9ae 100644
--- a/chrome/browser/password_manager/password_store_mac.cc
+++ b/chrome/browser/password_manager/password_store_mac.cc
@@ -866,7 +866,8 @@ PasswordStoreMac::PasswordStoreMac(
PasswordStoreMac::~PasswordStoreMac() {}
bool PasswordStoreMac::Init(
- const syncer::SyncableService::StartSyncFlare& flare) {
+ const syncer::SyncableService::StartSyncFlare& flare,
+ const std::string& sync_username) {
DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
thread_.reset(new base::Thread("Chrome_PasswordStore_Thread"));
@@ -874,7 +875,7 @@ bool PasswordStoreMac::Init(
thread_.reset(NULL);
return false;
}
- return password_manager::PasswordStore::Init(flare);
+ return password_manager::PasswordStore::Init(flare, sync_username);
}
void PasswordStoreMac::Shutdown() {
@@ -892,8 +893,8 @@ PasswordStoreMac::GetBackgroundTaskRunner() {
return (thread_.get()) ? thread_->message_loop_proxy() : NULL;
}
-void PasswordStoreMac::ReportMetricsImpl() {
- login_metadata_db_->ReportMetrics();
+void PasswordStoreMac::ReportMetricsImpl(const std::string& sync_username) {
+ login_metadata_db_->ReportMetrics(sync_username);
}
PasswordStoreChangeList PasswordStoreMac::AddLoginImpl(
« no previous file with comments | « chrome/browser/password_manager/password_store_mac.h ('k') | chrome/browser/password_manager/password_store_mac_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698