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

Unified Diff: chrome/browser/password_manager/chrome_password_manager_client.h

Issue 2302823003: Only report PasswordState in Sync for UMA+non-custom passphrase users. (Closed)
Patch Set: Split apart state-transition test Created 4 years, 3 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/chrome_password_manager_client.h
diff --git a/chrome/browser/password_manager/chrome_password_manager_client.h b/chrome/browser/password_manager/chrome_password_manager_client.h
index 8688ff83f261bb8cfb09c60243b393f6d25c79e8..c4a0e2a20a6e16319da6e1a9feade05fbe000698 100644
--- a/chrome/browser/password_manager/chrome_password_manager_client.h
+++ b/chrome/browser/password_manager/chrome_password_manager_client.h
@@ -165,6 +165,10 @@ class ChromePasswordManagerClient
bool one_local_credential,
const autofill::PasswordForm* form);
+ // Returns true if this profile has metrics reporting and active sync
+ // without custom sync passphrase.
+ static bool ShouldAnnotateNavigationEntries(Profile* profile);
+
Profile* const profile_;
password_manager::PasswordManager password_manager_;

Powered by Google App Engine
This is Rietveld 408576698