UKM that threads together multiple form interaction events.

tools/metrics/ukm/ukm.xml

Index: tools/metrics/ukm/ukm.xml
diff --git a/tools/metrics/ukm/ukm.xml b/tools/metrics/ukm/ukm.xml
index 75d8a1a22b2c52a6c3f111ac4eae536133bddb2c..09aec7366b9bc843f315c036142a52bf0cb34e16 100644
--- a/tools/metrics/ukm/ukm.xml
+++ b/tools/metrics/ukm/ukm.xml
@@ -38,7 +38,53 @@ along with a detailed description for each metric.
 
 <event name="Autofill.CardUploadDecision">
   <owner>sebsg@chromium.org</owner>
+  <summary>
+    Recorded when we decide whether to offer Card Upload to Payments.
+  </summary>
   <metric name="UploadDecision"/>
 </event>
 
+<event name="Autofill.DeveloperEngagement">
+  <owner>csashi@google.com</owner>
+  <summary>
+    Recorded when we parse a form.
+  </summary>
+  <metric name="DeveloperEngagement"/>
+</event>
+
+<event name="AutofillFormSubmittedState">
+  <owner>csashi@google.com</owner>
+  <metric name="AutofillFormSubmittedState"/>
+  <metric name="MillisecondsSinceFormLoaded"/>
+</event>
+
+<event name="FormEvent.Address">
+  <owner>csashi@google.com</owner>
+  <summary>
+    FormEvent for address/profile forms.
+  </summary>
+  <metric name="FormEvent.Address"/>
+  <metric name="MillisecondsSinceFormLoaded"/>
+</event>
+
+<event name="FormEvent.CreditCard">
+  <owner>csashi@google.com</owner>
+  <summary>
+    FormEvent for credit card forms.
+  </summary>
+  <metric name="FormEvent.CreditCard"/>
+  <metric name="MillisecondsSinceFormLoaded"/>
+</event>
+
+<event name="UnmaskPromptEvent">
+  <owner>csashi@google.com</owner>
+  <metric name="MillisecondsSinceFormLoaded"/>
+  <metric name="UnmaskPromptEvent"/>

[Steven Holte, 2017/04/12 02:26:31]

It seems like this is always AutoFill::UNMASK_PROMPT_SHOWN.  Is this needed?

[csashi, 2017/04/12 05:53:58]

Acknowledged. I was planning on adding other UnmaskPromptEvents too but they get
logged at different locations. To limit the scope of this change, I just logged
UNMASK_PROMPT_SHOWN, but it would be better to add it back when I have all the
unmask events. 

+</event>
+
+<event name="UserHappinessMetric">

[Steven Holte, 2017/04/12 02:26:31]

A lot of these metric/events names seem pretty ambiguous.  It would be helpful
if you describe these in terms of how they tell you what has happened on the
client.

[csashi, 2017/04/12 05:53:59]

Acknowledged. I will update up this after you review the changes to the
metrics/entries.

[csashi, 2017/04/12 18:54:17]

Done.

+  <metric name="MillisecondsSinceFormLoaded"/>
+  <metric name="UserHappinessMetric"/>
+</event>
+
 </ukm-configuration>