tools/metrics/actions/actions.xml - Issue 2220423002: Fix sync-credential-related metrics

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

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

Issue 2220423002: Fix sync-credential-related metrics (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Non-null driver + removing a forgotten LOG statement Created 4 years, 4 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

« components/password_manager/sync/browser/sync_credentials_filter_unittest.cc ('K') | « components/password_manager/sync/browser/sync_credentials_filter_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: tools/metrics/actions/actions.xml

diff --git a/tools/metrics/actions/actions.xml b/tools/metrics/actions/actions.xml

index 820924fbc9cea2e8b94f713f06231047d9da0abf..833b8b5b698da07cf954d8e7194ff61150cbb346 100644

--- a/tools/metrics/actions/actions.xml

+++ b/tools/metrics/actions/actions.xml

@@ -12389,6 +12389,16 @@ should be able to be added at any place in this file.

</description>

</action>

+<action name="PasswordManager_ProcessingUpdate">

+ <owner>vabr@chromium.org</owner>

+ <owner>vasilii@chromium.org</owner>

+ <description>

+ The user logged in successfully with a credential originally filled by

+ Chrome, and either did not change the stored password at all, or authorized

+ Chrome to updat it with the current password value.

vasilii 2016/08/08 15:34:24 update

vabr (Chromium) 2016/08/09 12:19:52 Done.

+ </description>

+</action>

<owner>Please list the metric's owners. Add more owner tags as needed.</owner>

<description>Please enter the description of this user action.</description>

@@ -12399,6 +12409,15 @@ should be able to be added at any place in this file.

<description>Please enter the description of this user action.</description>

</action>

+<action name="PasswordManager_SyncCredentialFilledAndLoginSuccessfull">

+ <owner>vabr@chromium.org</owner>

+ <owner>vasilii@chromium.org</owner>

+ <description>

+ Counts the number of times when Chrome fills the user's sync account

+ credentials, and the user logs in successfully after submitting them.

+ </description>

+</action>

<owner>gcasto@chromium.org</owner>

<owner>vabr@chromium.org</owner>

@@ -12415,6 +12434,7 @@ should be able to be added at any place in this file.

Counts the number of times a user submits an autofilled credential which

matches their sync account.

</description>

+ <obsolete>Counted the wrong thing, http://crbug.com/635428</obsolete>

</action>