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

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 2634163002: Fetch federated PSL-matches from the password store. (Closed)
Patch Set: Updated Histogram. Created 3 years, 11 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:
Download patch
« no previous file with comments | « components/password_manager/core/browser/psl_matching_helper.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index efdf0cbeee68325f1e3af8b63e4698ed970e899e..1f8ca93ebb5c822a1ed23363e7b52a70ff23032b 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -44581,8 +44581,11 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<owner>yfriedman@chromium.org</owner>
<summary>
Breakdown on trigger rate of providing a password form autofill entry based
- on matching stored information using the public suffix list for possible
- matches.
+ on matching stored information using the public suffix list (PSL) for
+ possible matches. In addition, this metric also counts cases where a
+ existing federated entry was successfully matched via PSL. For example, this
+ includes cases where an existing federated credential for
+ https://example.com was used for https://subdomain.example.com.
</summary>
</histogram>
@@ -99835,6 +99838,7 @@ value.
<int value="0" label="Matching not used"/>
<int value="1" label="No match"/>
<int value="2" label="Match"/>
+ <int value="3" label="Federated match"/>
Ilya Sherman 2017/01/19 02:24:27 Ah, I see, you are adding a bucket. This change m
</enum>
<enum name="PasswordManagerShowEmptyUsername" type="int">
« no previous file with comments | « components/password_manager/core/browser/psl_matching_helper.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698