Commit 92d4521b authored by Jan Wilken Dörrie's avatar Jan Wilken Dörrie Committed by Chromium LUCI CQ

[Passwords] Extend Histograms

This change extends several Password Manager histograms.

R=chromium-metrics-reviews@google.com

Fixed: 1165113
Change-Id: If442fb40bd477aa860a658441c7c433b6d156a6f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2636154
Commit-Queue: Mark Pearson <mpearson@chromium.org>
Reviewed-by: default avatarMark Pearson <mpearson@chromium.org>
Reviewed-by: default avatarMichael Lippautz <mlippautz@chromium.org>
Auto-Submit: Jan Wilken Dörrie <jdoerrie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844819}
parent df3725d4
...@@ -889,13 +889,13 @@ reviews. Googlers can read more about this at go/gwsq-gerrit. ...@@ -889,13 +889,13 @@ reviews. Googlers can read more about this at go/gwsq-gerrit.
</histogram> </histogram>
<histogram name="PasswordManager.CleanedUpPasswords" units="units" <histogram name="PasswordManager.CleanedUpPasswords" units="units"
expires_after="M90"> expires_after="M95">
<owner>vasilii@chromium.org</owner> <owner>vasilii@chromium.org</owner>
<owner>src/components/password_manager/OWNERS</owner> <owner>src/components/password_manager/OWNERS</owner>
<summary> <summary>
Collects positive number of undecryptable passwords that were successfully Collects positive number of undecryptable passwords that were successfully
deleted when syncing was enabled. Called for sync user for each profile if deleted when syncing was enabled. Recorded for sync users for each profile
there is at least one password deleted when merging logins from Sync and if there is at least one password deleted when merging logins from Sync and
local database. local database.
</summary> </summary>
</histogram> </histogram>
...@@ -1238,19 +1238,22 @@ reviews. Googlers can read more about this at go/gwsq-gerrit. ...@@ -1238,19 +1238,22 @@ reviews. Googlers can read more about this at go/gwsq-gerrit.
</histogram> </histogram>
<histogram name="PasswordManager.ImportedPasswordsPerUserInCSV" units="units" <histogram name="PasswordManager.ImportedPasswordsPerUserInCSV" units="units"
expires_after="M90"> expires_after="M95">
<owner>vasilii@chromium.org</owner> <owner>vasilii@chromium.org</owner>
<owner>src/components/password_manager/OWNERS</owner> <owner>src/components/password_manager/OWNERS</owner>
<summary>The number of passwords imported in CSV format per user</summary> <summary>
The number of passwords imported in CSV format per user. Recorded when the
user imports passwords within the password settings.
</summary>
</histogram> </histogram>
<histogram name="PasswordManager.ImportPasswordFromCSVResult" <histogram name="PasswordManager.ImportPasswordFromCSVResult"
enum="PasswordImportFromCSVResult" expires_after="M90"> enum="PasswordImportFromCSVResult" expires_after="M95">
<owner>vasilii@chromium.org</owner> <owner>vasilii@chromium.org</owner>
<owner>src/components/password_manager/OWNERS</owner> <owner>src/components/password_manager/OWNERS</owner>
<summary> <summary>
The response code of successful or failed password import attempt from a CSV The response code of successful or failed password import attempt from a CSV
file. file. Recorded when the user imports passwords within the password settings.
</summary> </summary>
</histogram> </histogram>
...@@ -1820,7 +1823,7 @@ reviews. Googlers can read more about this at go/gwsq-gerrit. ...@@ -1820,7 +1823,7 @@ reviews. Googlers can read more about this at go/gwsq-gerrit.
</histogram> </histogram>
<histogram name="PasswordManager.PasswordStoreInitResult" enum="BooleanSuccess" <histogram name="PasswordManager.PasswordStoreInitResult" enum="BooleanSuccess"
expires_after="M90"> expires_after="M95">
<owner>vasilii@chromium.org</owner> <owner>vasilii@chromium.org</owner>
<owner>src/components/password_manager/OWNERS</owner> <owner>src/components/password_manager/OWNERS</owner>
<summary> <summary>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment