Logs all reasons card upload was not offered in UKM and UMA.

components/autofill/core/browser/autofill_metrics.cc

Index: components/autofill/core/browser/autofill_metrics.cc
diff --git a/components/autofill/core/browser/autofill_metrics.cc b/components/autofill/core/browser/autofill_metrics.cc
index 77442395f201073a68feac7dc21890f545343393..20604947bb9ebcb8228e2d413c1ed70ee08a63e9 100644
--- a/components/autofill/core/browser/autofill_metrics.cc
+++ b/components/autofill/core/browser/autofill_metrics.cc
@@ -726,16 +726,18 @@ void AutofillMetrics::LogShowedHttpNotSecureExplanation() {
 }
 
 // static
-void AutofillMetrics::LogCardUploadDecisionUkm(
+void AutofillMetrics::LogCardUploadDecisionsUkm(
     ukm::UkmService* ukm_service,
     const GURL& url,
-    AutofillMetrics::CardUploadDecisionMetric upload_decision) {
-  if (upload_decision >= AutofillMetrics::NUM_CARD_UPLOAD_DECISION_METRICS)
-    return;
-
-  const std::vector<std::pair<const char*, int>> metrics = {
-      {internal::kUKMCardUploadDecisionMetricName,
-       static_cast<int>(upload_decision)}};
+    const std::vector<AutofillMetrics::CardUploadDecisionMetric>&
+        upload_decisions) {
+  std::vector<std::pair<const char*, int>> metrics;
+
+  for (const auto it : upload_decisions) {
+    DCHECK_LT(it, AutofillMetrics::NUM_CARD_UPLOAD_DECISION_METRICS);
+    metrics.push_back(
+        {internal::kUKMCardUploadDecisionMetricName, static_cast<int>(it)});

[sebsg, 2017/04/28 15:26:24]

This works, but what do you think about a bitfield? I feel like it would better
represent your data and maybe easier to manipulate? You'd be able to do
something like

upload_decision |= DECISION_NO_ADDRESS;

then later

upload_decision |= DECISION_NO_CVC

etc.

And then you can record as one metric :)

[csashi, 2017/04/28 23:47:45]

Done.

+  }
   LogUkm(ukm_service, url, internal::kUKMCardUploadDecisionEntryName, metrics);
 }