tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 51b2c69057a9778abc76969b5c093972495af944..d06fe94cfd7a2d9176266f6e4ea61264e6b9b077 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -41691,6 +41691,34 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="PaymentRequest.NumberOfSelectionAdds">
+  <owner>sebsg@chromium.org</owner>
+  <summary>
+    The number of times the user added an entry during a Payment Request.
+  </summary>
+</histogram>
+
+<histogram name="PaymentRequest.NumberOfSelectionChanges">
+  <owner>sebsg@chromium.org</owner>
+  <summary>
+    The number of times the user changed an entry during a Payment Request.
+  </summary>
+</histogram>
+
+<histogram name="PaymentRequest.NumberOfSelectionEdits">
+  <owner>sebsg@chromium.org</owner>
+  <summary>
+    The number of times the user edited an entry during a Payment Request.
+  </summary>
+</histogram>
+
+<histogram name="PaymentRequest.NumberOfSuggestionsShown">
+  <owner>sebsg@chromium.org</owner>
+  <summary>
+    The number of suggestions shown to the user during a payment request.
+  </summary>
+</histogram>
+
 <histogram name="PaymentRequest.RequestedInformation"
     enum="PaymentRequestRequestedInformation">
   <owner>sebsg@chromium.org</owner>
@@ -107466,6 +107494,47 @@ value.
   <affected-histogram name="PasswordManager.TotalAccountsHiRes.WithScheme"/>
 </histogram_suffixes>
 
+<histogram_suffixes name="PaymentRequestOutcome">
+  <suffix name="Completed" label="The Payment Request was completed"/>
+  <suffix name="UserAborted"
+      label="The Payment Request was aborted by the user"/>
+  <suffix name="OtherAborted"
+      label="The Payment Request was aborted but not but the user"/>
+  <affected-histogram name="PaymentRequest.NumberOfSelectionAdds_ContactInfo"/>
+  <affected-histogram name="PaymentRequest.NumberOfSelectionAdds_CreditCards"/>
+  <affected-histogram
+      name="PaymentRequest.NumberOfSelectionAdds_ShippingAddress"/>
+  <affected-histogram
+      name="PaymentRequest.NumberOfSelectionChanges_ContactInfo"/>
+  <affected-histogram
+      name="PaymentRequest.NumberOfSelectionChanges_CreditCards"/>
+  <affected-histogram
+      name="PaymentRequest.NumberOfSelectionChanges_ShippingAddress"/>
+  <affected-histogram name="PaymentRequest.NumberOfSelectionEdits_ContactInfo"/>
+  <affected-histogram name="PaymentRequest.NumberOfSelectionEdits_CreditCards"/>
+  <affected-histogram
+      name="PaymentRequest.NumberOfSelectionEdits_ShippingAddress"/>
+  <affected-histogram
+      name="PaymentRequest.NumberOfSuggestionsShown_ContactInfo"/>
+  <affected-histogram
+      name="PaymentRequest.NumberOfSuggestionsShown_CreditCards"/>
+  <affected-histogram
+      name="PaymentRequest.NumberOfSuggestionsShown_ShippingAddress"/>
+</histogram_suffixes>
+
+<histogram_suffixes name="PaymentRequestSection">
+  <suffix name="ContactInfo"
+      label="The contact info section of a Payment Request"/>

[jwd, 2016/10/07 20:37:13]

Nit: Maybe make it "For the..." for these labels?

[sebsg, 2016/10/07 21:08:18]

Done.

+  <suffix name="CreditCards"
+      label="The credit card section of a Payment Request"/>
+  <suffix name="ShippingAddress"
+      label="The shipping address section of a Payment Request"/>
+  <affected-histogram name="PaymentRequest.NumberOfSelectionAdds"/>
+  <affected-histogram name="PaymentRequest.NumberOfSelectionChanges"/>
+  <affected-histogram name="PaymentRequest.NumberOfSelectionEdits"/>
+  <affected-histogram name="PaymentRequest.NumberOfSuggestionsShown"/>
+</histogram_suffixes>
+
 <histogram_suffixes name="PerformanceMonitor" separator=".">
   <suffix name="BrowserProcess"/>
   <suffix name="GPUProcess"/>