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

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

Issue 644053003: [Password Manager] Add UMA stats for custom passphrase users. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments Created 6 years, 2 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 6bfc2d9c0db9e50c33becb0078f49792bb214916..85f3d11318e34893084c4430fbe9dc6605b1f288 100644
--- a/chrome/browser/password_manager/password_store_mac.cc
+++ b/chrome/browser/password_manager/password_store_mac.cc
@@ -875,8 +875,7 @@ PasswordStoreMac::PasswordStoreMac(
PasswordStoreMac::~PasswordStoreMac() {}
bool PasswordStoreMac::Init(
- const syncer::SyncableService::StartSyncFlare& flare,
- const std::string& sync_username) {
+ const syncer::SyncableService::StartSyncFlare& flare) {
DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
thread_.reset(new base::Thread("Chrome_PasswordStore_Thread"));
@@ -884,7 +883,7 @@ bool PasswordStoreMac::Init(
thread_.reset(NULL);
return false;
}
- return password_manager::PasswordStore::Init(flare, sync_username);
+ return password_manager::PasswordStore::Init(flare);
}
void PasswordStoreMac::Shutdown() {
@@ -902,8 +901,10 @@ PasswordStoreMac::GetBackgroundTaskRunner() {
return (thread_.get()) ? thread_->message_loop_proxy() : NULL;
}
-void PasswordStoreMac::ReportMetricsImpl(const std::string& sync_username) {
- login_metadata_db_->ReportMetrics(sync_username);
+void PasswordStoreMac::ReportMetricsImpl(const std::string& sync_username,
+ bool custom_passphrase_sync_enabled) {
+ login_metadata_db_->ReportMetrics(sync_username,
+ custom_passphrase_sync_enabled);
}
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