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

Unified Diff: components/password_manager/core/browser/password_manager_metrics_util.cc

Issue 1781633002: Mark PasswordManager.InfoBarResponse as obsolete (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@593295_remove_groups_code
Patch Set: Just rebased Created 4 years, 9 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/password_manager_metrics_util.cc
diff --git a/components/password_manager/core/browser/password_manager_metrics_util.cc b/components/password_manager/core/browser/password_manager_metrics_util.cc
index d26f8a922477edd2caf1938e3a8b886cd65d9157..f6aa33706d3b7809f72ee0bc6206f138c237bc04 100644
--- a/components/password_manager/core/browser/password_manager_metrics_util.cc
+++ b/components/password_manager/core/browser/password_manager_metrics_util.cc
@@ -31,28 +31,6 @@ void LogUMAHistogramBoolean(const std::string& name, bool sample) {
histogram->AddBoolean(sample);
}
-void LogUIDismissalReason(ResponseType type) {
- UIDismissalReason reason = NO_DIRECT_INTERACTION;
- switch (type) {
- case NO_RESPONSE:
- reason = NO_DIRECT_INTERACTION;
- break;
- case REMEMBER_PASSWORD:
- reason = CLICKED_SAVE;
- break;
- case NEVER_REMEMBER_PASSWORD:
- reason = CLICKED_NEVER;
- break;
- case INFOBAR_DISMISSED:
- reason = CLICKED_CANCEL;
- break;
- case NUM_RESPONSE_TYPES:
- NOTREACHED();
- break;
- }
- LogUIDismissalReason(reason);
-}
-
void LogUIDismissalReason(UIDismissalReason reason) {
UMA_HISTOGRAM_ENUMERATION("PasswordManager.UIDismissalReason",
reason,

Powered by Google App Engine
This is Rietveld 408576698